Merge branch i/6202-remove-env.isEdge to master.

Internal: Removed mentions on Edge. See #6202.
ckeditor · Apr 20, 2020 · 446170a · 446170a
2 parents bfe19b3 + 40c23ca
commit 446170a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/ckeditor5-engine/src/view/domconverter.js b/packages/ckeditor5-engine/src/view/domconverter.js
@@ -169,8 +169,7 @@ export default class DomConverter {
 			this._domToViewMapping.delete( domElement );
 			this._viewToDomMapping.delete( viewElement );
 
-			// Use Array.from because of MS Edge (#923).
-			for ( const child of Array.from( domElement.childNodes ) ) {
+			for ( const child of domElement.childNodes ) {
 				this.unbindDomElement( child );
 			}
 		}

diff --git a/packages/ckeditor5-engine/tests/view/renderer.js b/packages/ckeditor5-engine/tests/view/renderer.js
@@ -2056,7 +2056,7 @@ describe( 'Renderer', () => {
 
 			// Use a forgiving way of checking what the selection contains
 			// because Safari normalizes the selection ranges so precise checking is troublesome.
-			// Also, Edge returns a normal space instead of nbsp so we need to use even more alternatives.
+			// Also, Firefox returns a normal space instead of nbsp so we need to use even more alternatives.
 			function assertDomSelectionContents( domSelection, expectedContainer, expectedText ) {
 				const domSelectionContainer = domSelection.getRangeAt( 0 ).commonAncestorContainer;