Omnibox refactor, introduced OmniboxController.

chrome/browser/ui/omnibox/omnibox_controller.cc

+// 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.
+
+#include "chrome/browser/ui/omnibox/omnibox_controller.h"
+
+#include "chrome/browser/autocomplete/autocomplete_classifier.h"
+#include "chrome/browser/autocomplete/autocomplete_controller.h"
+#include "chrome/browser/search/search.h"
+#include "chrome/browser/ui/omnibox/omnibox_edit_model.h"
+

[Peter Kasting, 2013/04/25 18:24:55]

Tiny nit: I've slowly been moving towards two newlines between the #includes and
code (or between one class and the next; i.e. major divisions within the file).

[beaudoin, 2013/04/25 20:34:51]

Happy to adopt this. I've done it in the tests too, but it looks weird in the
.h. If you see inconsistencies anywhere let me know.

+OmniboxController::OmniboxController(OmniboxEditModel* omnibox_edit_model,
+                                     Profile* profile)
+    : omnibox_edit_model_(omnibox_edit_model) {
+  // Use a restricted subset of the autocomplete providers if we're using the
+  // Instant Extended API, as it doesn't support them all.

[Peter Kasting, 2013/04/25 18:24:55]

Nit: Comment is unnecessary

[beaudoin, 2013/04/25 20:34:51]

Done.

+  autocomplete_controller_.reset(new AutocompleteController(profile, this,
+      chrome::IsInstantExtendedAPIEnabled() ?
+          AutocompleteClassifier::kInstantExtendedOmniboxProviders :
+          AutocompleteClassifier::kDefaultOmniboxProviders));
+}
+
+OmniboxController::~OmniboxController() {
+}
+
+void OmniboxController::OnResultChanged(bool default_match_changed) {
+  omnibox_edit_model_->OnResultChanged(default_match_changed);
+}