Fixes #4393: Ensure clipboard gets the content from the latest selection (flush pending events)

src/vs/editor/browser/controller/keyboardHandler.ts

@@ -207,7 +207,7 @@ export class KeyboardHandler extends ViewEventHandler implements IDisposable {
 		this.contentWidth = 0;
 		this.scrollLeft = 0;
 
-		this.textAreaHandler = new TextAreaHandler(browser, this._getStrategy(), this.textArea, this.context.model);
+		this.textAreaHandler = new TextAreaHandler(browser, this._getStrategy(), this.textArea, this.context.model, () => this.viewHelper.flushAnyAccumulatedEvents());
 
 		this._toDispose = [];
 		this._toDispose.push(this.textAreaHandler.onKeyDown((e) => this.viewController.emitKeyDown(<IKeyboardEvent>e._actual)));

src/vs/editor/browser/editorBrowser.ts

@@ -42,6 +42,7 @@ export interface IKeyboardHandlerHelper {
 	viewDomNode:HTMLElement;
 	textArea:HTMLTextAreaElement;
 	visibleRangeForPositionRelativeToEditor(lineNumber:number, column:number): editorCommon.VisibleRange;
+	flushAnyAccumulatedEvents(): void;
 }
 
 export interface IPointerHandlerHelper {

src/vs/editor/browser/view/viewImpl.ts

@@ -399,6 +399,9 @@ export class View extends ViewEventHandler implements editorBrowser.IView, IDisp
 					return null;
 				}
 				return visibleRanges[0];
+			},
+			flushAnyAccumulatedEvents: () => {
+				this._flushAnyAccumulatedEvents();
 			}
 		};
 	}

src/vs/editor/common/controller/textAreaHandler.ts

@@ -62,6 +62,7 @@ export class TextAreaHandler extends Disposable {
 	private Browser:IBrowser;
 	private textArea:ITextAreaWrapper;
 	private model:ISimpleModel;
+	private flushAnyAccumulatedEvents:()=>void;
 
 	private selection:IEditorRange;
 	private selections:IEditorRange[];
@@ -80,11 +81,12 @@ export class TextAreaHandler extends Disposable {
 
 	private _nextCommand: ReadFromTextArea;
 
-	constructor(Browser:IBrowser, strategy:TextAreaStrategy, textArea:ITextAreaWrapper, model:ISimpleModel) {
+	constructor(Browser:IBrowser, strategy:TextAreaStrategy, textArea:ITextAreaWrapper, model:ISimpleModel, flushAnyAccumulatedEvents:()=>void) {
 		super();
 		this.Browser = Browser;
 		this.textArea = textArea;
 		this.model = model;
+		this.flushAnyAccumulatedEvents = flushAnyAccumulatedEvents;
 		this.selection = new Range(1, 1, 1, 1);
 		this.selections = [new Range(1, 1, 1, 1)];
 		this.cursorPosition = new Position(1, 1);
@@ -181,11 +183,15 @@ export class TextAreaHandler extends Disposable {
 		// --- Clipboard operations
 
 		this._register(this.textArea.onCut((e) => {
+			// Ensure we have the latest selection => ask all pending events to be sent
+			this.flushAnyAccumulatedEvents();
 			this._ensureClipboardGetsEditorSelection(e);
 			this.asyncTriggerCut.schedule();
 		}));
 
 		this._register(this.textArea.onCopy((e) => {
+			// Ensure we have the latest selection => ask all pending events to be sent
+			this.flushAnyAccumulatedEvents();
 			this._ensureClipboardGetsEditorSelection(e);
 		}));