Fixes #4313: push undo/redo stack element before and after `editor.executeEdits`

src/vs/editor/common/commonCodeEditor.ts

@@ -608,9 +608,11 @@ export abstract class CommonCodeEditor extends EventEmitter implements IActionPr
 			// read only editor => sorry!
 			return false;
 		}
+		this.model.pushStackElement();
 		this.model.pushEditOperations(this.cursor.getSelections(), edits, () => {
 			return this.cursor.getSelections();
 		});
+		this.model.pushStackElement();
 		return true;
 	}
 

src/vs/editor/contrib/snippet/common/snippet.ts

@@ -794,9 +794,7 @@ class SnippetController implements ISnippetController {
 		SnippetController._addCommandForSnippet(this._editor.getModel(), prepared.adaptedSnippet, prepared.typeRange, edits);
 
 		if (edits.length > 0) {
-			this._editor.getModel().pushStackElement();
 			this._editor.executeEdits('editor.contrib.insertSnippetHelper', edits);
-			this._editor.getModel().pushStackElement();
 		}
 
 		let cursorOnly = SnippetController._getSnippetCursorOnly(prepared.adaptedSnippet);
@@ -820,9 +818,7 @@ class SnippetController implements ISnippetController {
 		}
 
 		if (edits.length > 0) {
-			this._editor.getModel().pushStackElement();
 			this._editor.executeEdits('editor.contrib.insertSnippetHelper', edits);
-			this._editor.getModel().pushStackElement();
 		}
 	}