commit 33c72e13ece49c36938905cb6d87fbf15e9a0cfb
author Kevin Bailey <krb@chromium.org> Thu Jan 17 02:24:13 2019
committer Commit Bot <commit-bot@chromium.org> Thu Jan 17 02:24:13 2019
tree 2b448ddba65a748278985f30507b6b8f01c494fa
parent 10051dfc9a7ae9a9534a238ea46748230b43f370

[omnibox] Condition accessibility call

An added call is apparently triggering IME misbehavior. Condition the
call to only when it is needed.

Bug: 896962
Change-Id: I9bfb9aa702d5860f5274c78f9e8f0bb37109f685
Reviewed-on: https://chromium-review.googlesource.com/c/1413882
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Kevin Bailey <krb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623521}

components/omnibox/browser/omnibox_popup_model.cc

diff --git a/components/omnibox/browser/omnibox_popup_model.cc b/components/omnibox/browser/omnibox_popup_model.cc
index d99d8dfe..f0ef0ec 100644
--- a/components/omnibox/browser/omnibox_popup_model.cc
+++ b/components/omnibox/browser/omnibox_popup_model.cc
@@ -208,9 +208,11 @@
   selected_line_state_ = state;
   view_->InvalidateLine(selected_line_);
 
-  // Ensures update of accessibility data.
-  edit_model_->view()->OnTemporaryTextMaybeChanged(
-    edit_model_->view()->GetText(), match, false, false);
+  // Ensures update of accessibility data for button text.
+  if (state == BUTTON_FOCUSED) {
+    edit_model_->view()->OnTemporaryTextMaybeChanged(
+        edit_model_->view()->GetText(), match, false, false);
+  }
 }
 
 void OmniboxPopupModel::TryDeletingCurrentItem() {