[omnibox] Encapsulate an accessibility call

After a previous change, it was apparent that OmniboxEditModel was
exposing more members than it needed to, and that the code was a
little more complicated than necessary. This CL encapsulates the
call, so that no data needs to be public, and the code is simpler.

Change-Id: Ibb3dcc23148d1e2568f54a9645de4c4267e3d2b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903619Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Kevin Bailey <krb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713703}

components/omnibox/browser/omnibox_edit_model.cc

@@ -1475,6 +1475,10 @@ void OmniboxEditModel::OnCurrentMatchChanged() {
 const char OmniboxEditModel::kCutOrCopyAllTextHistogram[] =
     "Omnibox.CutOrCopyAllText";
 
+void OmniboxEditModel::SetAccessibilityLabel(const AutocompleteMatch& match) {
+  view_->SetAccessibilityLabel(view_->GetText(), match);
+}
+
 bool OmniboxEditModel::PopupIsOpen() const {
   return popup_model() && popup_model()->IsOpen();
 }

components/omnibox/browser/omnibox_edit_model.h

@@ -385,7 +385,8 @@ class OmniboxEditModel {
   // Name of the histogram tracking cut or copy omnibox commands.
   static const char kCutOrCopyAllTextHistogram[];
 
-  OmniboxView* view() { return view_; }
+  // Just forwards the call to the OmniboxView referred within.
+  void SetAccessibilityLabel(const AutocompleteMatch& match);
 
  private:
   friend class OmniboxControllerTest;

components/omnibox/browser/omnibox_popup_model.cc

@@ -195,8 +195,7 @@ void OmniboxPopupModel::SetSelectedLineState(LineState state) {
   view_->InvalidateLine(selected_line_);
 
   if (state == BUTTON_FOCUSED) {
-    edit_model_->view()->SetAccessibilityLabel(edit_model_->view()->GetText(),
-                                               match);
+    edit_model_->SetAccessibilityLabel(match);
     view_->ProvideButtonFocusHint(selected_line_);
   }
 }