Centralize logic around Instant modes.

chrome/browser/ui/omnibox/omnibox_edit_model.cc

Index: chrome/browser/ui/omnibox/omnibox_edit_model.cc
diff --git a/chrome/browser/ui/omnibox/omnibox_edit_model.cc b/chrome/browser/ui/omnibox/omnibox_edit_model.cc
index 3b1cd78c544051287ce9a022468e35901a3abe5e..6c679afacf43ea58fe5fdcd908c755a15058fa76 100644
--- a/chrome/browser/ui/omnibox/omnibox_edit_model.cc
+++ b/chrome/browser/ui/omnibox/omnibox_edit_model.cc
@@ -430,6 +430,11 @@ void OmniboxEditModel::StartAutocomplete(
   popup_->SetHoveredLine(OmniboxPopupModel::kNoMatch);
   // We don't explicitly clear OmniboxPopupModel::manually_selected_match, as
   // Start ends up invoking OmniboxPopupModel::OnResultChanged which clears it.
+  SearchProvider* search_provider = autocomplete_controller_->search_provider();
+  if (search_provider) {
+    search_provider->set_instant_suggest_enabled(controller_->GetInstant() &&

[sky, 2012/08/24 19:11:52]

Don't you need to NULL check instant?

[sreeram, 2012/08/24 19:15:29]

Correct. Isn't that what the "&&" is doing? GetInstant() returns an
InstantController*, not a scoped_ptr or such, if that's what you were wondering.

+        controller_->GetInstant()->IsSuggestEnabled());
+  }
   autocomplete_controller_->Start(
       user_text_, GetDesiredTLD(),
       prevent_inline_autocomplete || just_deleted_text_ ||