corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1323429 - [1.1] Don't verify IME root content with e10s enabled. r=jchen

Browse Source
This commit is contained in:
Eugen Sawin
2017-01-24 23:59:54 +01:00
parent 7cacb219a7
commit 206d6a12e8
1 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
widget/android/nsWindow.cpp
View File

@@ -2763,12 +2763,19 @@ nsWindow::GeckoViewSupport::FlushIMEChanges(FlushChangesFlag aFlags)
nsCOMPtr<nsIContent> imeRoot;
// If we are receiving notifications, we must have selection/root content.
MOZ_ALWAYS_SUCCEEDS(IMEStateManager::GetFocusSelectionAndRoot(
getter_AddRefs(imeSelection), getter_AddRefs(imeRoot)));
nsresult rv = IMEStateManager::GetFocusSelectionAndRoot(
getter_AddRefs(imeSelection), getter_AddRefs(imeRoot));
// With e10s enabled, GetFocusSelectionAndRoot will fail because the IME
// content observer is out of process.
const bool e10sEnabled = rv == NS_ERROR_NOT_AVAILABLE;
if (!e10sEnabled) {
MOZ_ALWAYS_SUCCEEDS(rv);
}
// Make sure we still have a valid selection/root. We can potentially get
// a stale selection/root if the editor becomes hidden, for example.
NS_ENSURE_TRUE_VOID(imeRoot->IsInComposedDoc());
NS_ENSURE_TRUE_VOID(e10sEnabled || imeRoot->IsInComposedDoc());
RefPtr<nsWindow> kungFuDeathGrip(&window);
window.UserActivity();
@@ -2817,7 +2824,7 @@ nsWindow::GeckoViewSupport::FlushIMEChanges(FlushChangesFlag aFlags)
change.mNewEnd - change.mStart);
window.DispatchEvent(&event);
NS_ENSURE_TRUE_VOID(event.mSucceeded);
NS_ENSURE_TRUE_VOID(event.mReply.mContentsRoot == imeRoot.get());
NS_ENSURE_TRUE_VOID(e10sEnabled || event.mReply.mContentsRoot == imeRoot.get());
}
if (shouldAbort()) {
@@ -2838,7 +2845,7 @@ nsWindow::GeckoViewSupport::FlushIMEChanges(FlushChangesFlag aFlags)
window.DispatchEvent(&event);
NS_ENSURE_TRUE_VOID(event.mSucceeded);
NS_ENSURE_TRUE_VOID(event.mReply.mContentsRoot == imeRoot.get());
NS_ENSURE_TRUE_VOID(e10sEnabled || event.mReply.mContentsRoot == imeRoot.get());
if (shouldAbort()) {
return;