Omnibox refactor, moved OnResultChanged to OmniboxController

chrome/browser/ui/omnibox/omnibox_controller.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_OMNIBOX_OMNIBOX_CONTROLLER_H_
 #define CHROME_BROWSER_UI_OMNIBOX_OMNIBOX_CONTROLLER_H_
 
+#include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/autocomplete/autocomplete_controller_delegate.h"
 
 class AutocompleteController;
+struct AutocompleteMatch;
+class AutocompleteResult;
+class GURL;
 class OmniboxEditModel;
+class OmniboxPopupModel;
 class Profile;
 
 // This class controls the various services that can modify the content
 // for the omnibox, including AutocompleteController and InstantController. It
 // is responsible of updating the omnibox content.
 // TODO(beaudoin): Keep on expanding this class so that OmniboxEditModel no
 //     longer needs to hold any reference to AutocompleteController. Also make
 //     this the point of contact between InstantController and OmniboxEditModel.
 //     As the refactor progresses, keep the class comment up-to-date to
 //     precisely explain what this class is doing.
 class OmniboxController : public AutocompleteControllerDelegate {
 
  public:
   OmniboxController(OmniboxEditModel* omnibox_edit_model, Profile* profile);
   virtual ~OmniboxController();
 
   // AutocompleteControllerDelegate:
   virtual void OnResultChanged(bool default_match_changed) OVERRIDE;
 
   AutocompleteController* autocomplete_controller() {
     return autocomplete_controller_.get();
   }
 
+  void set_popup_model(OmniboxPopupModel* popup_model) {
+    popup_ = popup_model;
+  }
+
+  // TODO: The edit and popup should be siblings owned by the LocationBarView,

[Mathieu, 2013/05/03 19:09:02]

nit: the edit and popup what? be clearer, as it could be model, view,
controller...

[beaudoin, 2013/05/04 03:06:51]

Done.

+  // making this accessor unnecessary.
+  OmniboxPopupModel* popup_model() const { return popup_; }
+
+  // Turns off keyword mode for the current match.
+  void ClearPopupKeywordMode() const;
+
+  void InfoForCurrentSelection(AutocompleteMatch* match,
+                               GURL* alternate_nav_url) const;
+
+  const AutocompleteResult& result() const;
+
+  // TODO(beaudoin): Make private once OmniboxEditModel no longer refers to it.
+  void DoPreconnect(const AutocompleteMatch& match);
+
  private:
   // Weak, it owns us.
   // TODO(beaudoin): Consider defining a delegate to ease unit testing.
   OmniboxEditModel* omnibox_edit_model_;
 
+  Profile* profile_;
+
+  OmniboxPopupModel* popup_;
+
   scoped_ptr<AutocompleteController> autocomplete_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(OmniboxController);
 };
 
 #endif  // CHROME_BROWSER_UI_OMNIBOX_OMNIBOX_CONTROLLER_H_