Commit 9bff5bdc authored by michaelpg's avatar michaelpg Committed by Commit bot

Don't re-focus Settings window when opening IME options

These five lines just cause the current browser window to be shown after
the tab with the IME options has already been focused and activated in
another window.

BUG=409270
R=stevenjb@chromium.org,shuchen@chromium.org

Review URL: https://codereview.chromium.org/599663003

Cr-Commit-Position: refs/heads/master@{#296537}
parent 0cf13c01
...@@ -243,10 +243,6 @@ void CrosLanguageOptionsHandler::InputMethodOptionsOpenCallback( ...@@ -243,10 +243,6 @@ void CrosLanguageOptionsHandler::InputMethodOptionsOpenCallback(
ui::PAGE_TRANSITION_LINK, ui::PAGE_TRANSITION_LINK,
false); false);
browser->OpenURL(params); browser->OpenURL(params);
browser->window()->Show();
content::WebContents* web_contents =
browser->tab_strip_model()->GetActiveWebContents();
web_contents->GetDelegate()->ActivateContents(web_contents);
} }
void CrosLanguageOptionsHandler::AddImeProvider(base::ListValue* list) { void CrosLanguageOptionsHandler::AddImeProvider(base::ListValue* list) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment