Bug 622371 - Only set the selection to the beginning of the editable document if the editor is being initialized by entering designMode; r=bzbarsky a=blocking-final+

This commit is contained in:
Ehsan Akhgari 2011-01-06 17:13:51 -05:00
parent 00cba183fb
commit d48edb3f7d
3 changed files with 43 additions and 1 deletions

View File

@ -3308,7 +3308,7 @@ nsHTMLDocument::EditingStateChanged()
// If we're entering the design mode, put the selection at the beginning of
// the document for compatibility reasons.
if (designMode) {
if (designMode && oldState == eOff) {
rv = editor->BeginningOfDocument();
NS_ENSURE_SUCCESS(rv, rv);
}

View File

@ -70,6 +70,7 @@ _TEST_FILES = \
test_bug611182.html \
test_bug612447.html \
test_bug620906.html \
test_bug622371.html \
test_CF_HTML_clipboard.html \
test_contenteditable_focus.html \
test_contenteditable_text_input_handling.html \

View File

@ -0,0 +1,41 @@
<!DOCTYPE HTML>
<html>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=622371
-->
<head>
<title>Test for Bug 622371</title>
<script type="application/javascript" src="/MochiKit/packed.js"></script>
<script type="application/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"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=622371">Mozilla Bug 622371</a>
<p id="display"></p>
<div id="content">
<iframe src="data:text/html,<body contenteditable>abc</body>"></iframe>
</div>
<pre id="test">
<script type="application/javascript">
/** Test for Bug 622371 **/
SimpleTest.waitForExplicitFinish();
addLoadEvent(function() {
var i = document.querySelector("iframe");
var sel = i.contentWindow.getSelection();
var doc = i.contentDocument;
var body = doc.body;
i.focus();
sel.collapse(body, 1);
doc.designMode = "on";
doc.designMode = "off";
is(sel.getRangeAt(0).startOffset, 1, "The start offset of the selection shouldn't change");
is(sel.getRangeAt(0).endOffset, 1, "The end offset of the selection shouldn't change");
SimpleTest.finish();
});
</script>
</pre>
</body>
</html>