Bug 1151186 part.1 Don't reset IME enabled state after focus is changed by JS during setting focus r=enndeakin

This commit is contained in:
Masayuki Nakano 2015-04-08 11:50:12 +09:00
parent 32393dc505
commit f24ba16bfb
3 changed files with 51 additions and 3 deletions

View File

@ -1929,9 +1929,13 @@ nsFocusManager::Focus(nsPIDOMWindow* aWindow,
}
}
nsPresContext* presContext = presShell->GetPresContext();
IMEStateManager::OnChangeFocus(presContext, nullptr,
GetFocusMoveActionCause(aFlags));
if (!mFocusedContent) {
// When there is no focused content, IMEStateManager needs to adjust IME
// enabled state with the document.
nsPresContext* presContext = presShell->GetPresContext();
IMEStateManager::OnChangeFocus(presContext, nullptr,
GetFocusMoveActionCause(aFlags));
}
if (!aWindowRaised)
aWindow->UpdateCommands(NS_LITERAL_STRING("focus"), nullptr, 0);

View File

@ -16,6 +16,7 @@ support-files = window_bug478536.xul
support-files = window_bug538242.xul
[test_bug593307.xul]
support-files = window_bug593307_offscreen.xul window_bug593307_centerscreen.xul
[test_bug1151186.html]
[test_keycodes.xul]
[test_wheeltransaction.xul]
support-files = window_wheeltransaction.xul

View File

@ -0,0 +1,43 @@
<!DOCTYPE HTML>
<html>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=1151186
-->
<head>
<meta charset="utf-8">
<title>Test for Bug 1151186</title>
<script type="application/javascript" src="chrome://mochikit/content/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="chrome://global/skin"/>
<link rel="stylesheet" type="text/css" href="chrome://mochikit/content/tests/SimpleTest/test.css"/>
<script type="application/javascript">
/** Test for Bug 1151186 **/
SimpleTest.waitForExplicitFinish();
document.addEventListener("focus", () => {
document.getElementById("editor").focus();
SimpleTest.executeSoon(runTests);
});
function runTests()
{
is(document.activeElement, document.getElementById("editor"),
"The div element should be focused");
var utils = SpecialPowers.getDOMWindowUtils(window);
is(utils.IMEStatus, utils.IME_STATUS_ENABLED,
"IME should be enabled");
SimpleTest.finish();
}
</script>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1151186">Mozilla Bug 1151186</a>
<p id="display"></p>
<div id="content" style="display: none">
</div>
<div id="editor" contenteditable="true"></div>
<pre id="test">
</pre>
</body>
</html>