diff --git a/editor/libeditor/tests/mochitest.ini b/editor/libeditor/tests/mochitest.ini index 33b1648192..39245ecb40 100644 --- a/editor/libeditor/tests/mochitest.ini +++ b/editor/libeditor/tests/mochitest.ini @@ -224,6 +224,7 @@ skip-if = toolkit == 'android' skip-if = toolkit == 'android' # bug 1315898 [test_bug1314790.html] [test_bug1315065.html] +[test_bug1318312.html] [test_bug1328023.html] [test_bug1330796.html] [test_bug1332876.html] diff --git a/editor/libeditor/tests/test_bug1318312.html b/editor/libeditor/tests/test_bug1318312.html new file mode 100644 index 0000000000..e8bdba5233 --- /dev/null +++ b/editor/libeditor/tests/test_bug1318312.html @@ -0,0 +1,257 @@ + + + + + + + + + + + + + + + +mozilla-central: editor/libeditor/tests/test_bug1318312.html@22e1a32de5b88f1f56a156aefa95614b5cda204c + + + + + + + + + + +
+ +Mercurial > mozilla-central / file revision / editor/libeditor/tests/test_bug1318312.html@22e1a32de5b88f1f56a156aefa95614b5cda204c +
+ +
+
+summary | +shortlog | +changelog | +pushlog | +graph | +tags | +bookmarks | +branches | +files | +changeset | +file | +latest | +revisions | +annotate | +diff | +comparison | +raw | +help +
+ +
+
+ + +
Find changesets by keywords (author, files, the commit message), revision +number or hash, or revset expression.
+
+
+
+ +
editor/libeditor/tests/test_bug1318312.html
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + +
authorMasayuki Nakano <masayuki@d-toybox.com>
Tue, 14 Mar 2017 10:36:21 +0900
changeset 34744722e1a32de5b88f1f56a156aefa95614b5cda204c
child 403604 +b1aba58ebcddae563f4b949d357c9193b2d78897
permissions-rw-r--r--
+
+ +
Bug 1318312 part.3 Selection should move focus at every selection change when it's called by JS r=smaug + +Selection may be changed by methods of Selection or methods of Range retrieved by Selection.getRangeAt(). Selection::NotifySelectionListeners() is called after every selection change of each of them, so, this method must be a good point to move focus. + +If new common ancestor of all ranges is editable and in an editing host, we should move focus to it. Otherwise, if an editing host has focus but new common ancestor is not editable, we should move focus from the editing host. + +For consistency with the other browsers, this patch doesn't move focus to other focusable element. + +MozReview-Commit-ID: 6sNsuzwqECX
+ +
+
+
+<!DOCTYPE>
+<html>
+<head>
+<title>Test for bug1318312</title>
+<script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
+<script type="text/javascript" src="/tests/SimpleTest/EventUtils.js"></script>
+<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
+<style type="text/css">
+</style>
+</head>
+<body>
+<div id="outerEditor" contenteditable><p>content of outer editor</p>
+<div id="staticInEditor" contenteditable="false"><p>static content of outer editor</p>
+<div id="innerEditor" contenteditable><p>content of inner editor</p></div></div></div>
+<pre id="test">
+<script class="testbody" type="text/javascript">
+
+var outerEditor = document.getElementById("outerEditor");
+
+function runTests()
+{
+  outerEditor.focus();
+  is(document.activeElement, outerEditor,
+     "outerEditor should have focus");
+
+  // Move cursor into the innerEditor with ArrowDown key.  Then, focus shouldn't
+  // be moved to innerEditor from outerEditor.
+  // Note that Chrome moves focus in this case.  However, we should do that
+  // at least for now because user can move focus with ArrowUp key even from
+  // innerEditor to outerEditor but we don't allow such navigation.
+  // FYI: Edge behaves same as us.
+  synthesizeKey("KEY_ArrowDown", { code: "ArrowDown" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by moving caret");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+  is(document.getSelection().focusOffset, 0,
+     "focus offset of selection should be 0");
+
+  synthesizeKey("a", { code: "KeyA" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by typing a character");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+  is(document.getSelection().focusOffset, 1,
+     "focus offset of selection should be 1");
+
+  synthesizeKey("KEY_Enter", { code: "Enter" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by typing Enter");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").childNodes.item(1).firstChild,
+     "focus node of selection should be the text node in the second paragraph in the innerEditor");
+  is(document.getSelection().focusOffset, 0,
+     "focus offset of selection should be 0");
+
+  synthesizeKey("KEY_Backspace", { code: "Backspace" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by typing Backspace (removing the line breaker)");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+  is(document.getSelection().focusOffset, 1,
+     "focus offset of selection should be 1");
+
+  synthesizeKey("KEY_ArrowLeft", { code: "ArrowLeft", shiftKey: true });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by typing Shift+ArrowLeft (selecting 'a')");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+  is(document.getSelection().focusOffset, 0,
+     "focus offset of selection should be 0");
+  is(document.getSelection().anchorNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "anchor node of selection should be the text node in the innerEditor");
+  is(document.getSelection().anchorOffset, 1,
+     "anchor offset of selection should be 1");
+
+  synthesizeKey("KEY_Delete", { code: "Delete" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by typing Delete (removing the 'a')");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+  is(document.getSelection().focusOffset, 0,
+     "focus offset of selection should be 0");
+
+  synthesizeKey("KEY_ArrowUp", { code: "ArrowUp" });
+  is(document.activeElement, outerEditor,
+     "outerEditor should still have focus because focus shouldn't be moved by moving caret from innerEditor to outerEditor");
+  is(document.getSelection().focusNode, document.getElementById("outerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the outerEditor");
+  is(document.getSelection().focusOffset, 0,
+     "focus offset of selection should be 0");
+
+  // However, clicking in innerEditor should move focus.
+  synthesizeMouseAtCenter(innerEditor, {});
+  is(document.activeElement, innerEditor,
+     "innerEditor should get focus because focus should be moved to innerEditor even from outerEditor by click");
+  is(document.getSelection().focusNode, document.getElementById("innerEditor").firstChild.firstChild,
+     "focus node of selection should be the text node in the innerEditor");
+
+  SimpleTest.finish();
+}
+
+SimpleTest.waitForExplicitFinish();
+SimpleTest.waitForFocus(runTests);
+
+</script>
+</pre>
+</body>
+</html>
+
+
+ + + +
+
mozilla-central
+
Deployed from 20cb7a61bef3 at 2023-12-13T16:39:00Z.
+ +
+ +
+ + + diff --git a/editor/libeditor/tests/test_bug676401.html b/editor/libeditor/tests/test_bug676401.html index aa468fdc6c..2091c7957b 100644 --- a/editor/libeditor/tests/test_bug676401.html +++ b/editor/libeditor/tests/test_bug676401.html @@ -43,8 +43,15 @@ var alwaysEnabledCommands = [ "styleWithCSS", ]; +function ensureNobodyHasFocus() +{ + document.activeElement.blur(); +} + function IsCommandEnabled(command) { var enabled; + + ensureNobodyHasFocus(); // non-editable div: should return false unless alwaysEnabled window.getSelection().selectAllChildren(gBlock1); @@ -90,6 +97,8 @@ function runTests() { IsCommandEnabled(commands[i]); // These are privileged, and available only to chrome. + ensureNobodyHasFocus(); + window.getSelection().selectAllChildren(gBlock2); commands = ["paste"]; for (i = 0; i < commands.length; i++) { is(document.queryCommandEnabled(commands[i]), false, diff --git a/editor/libeditor/tests/test_contenteditable_focus.html b/editor/libeditor/tests/test_contenteditable_focus.html index 051ac7b2f1..2c810051d8 100644 --- a/editor/libeditor/tests/test_contenteditable_focus.html +++ b/editor/libeditor/tests/test_contenteditable_focus.html @@ -134,14 +134,14 @@ function runTestsInternal() otherEditor.focus(); is(SpecialPowers.unwrap(fm.focusedElement), otherEditor, "the other editor didn't get focus #2"); - // Set selection to the span element in the editor (unfocused) + // Set selection to the span element in the editor. range = document.createRange(); range.setStart(spanInEditor.firstChild, 5); selection.removeAllRanges(); selection.addRange(range); is(selection.rangeCount, 1, "selection range is lost from the document"); - is(SpecialPowers.unwrap(fm.focusedElement), otherEditor, - "the other editor shouldn't lose focus by selection range change"); + is(SpecialPowers.unwrap(fm.focusedElement), editor, + "the other editor should lose focus by selection range change"); ok(selCon.caretVisible, "caret isn't visible in inputTextInEditor"); // Move focus to the editor editor.focus();