Create a new helper function to extract search terms from the URL irrespective of the availablility

chrome/browser/ui/browser_instant_controller.cc

 // Copyright 2012 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/browser_instant_controller.h"
 
 #include "base/bind.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_web_ui.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 55 matching lines @@
   // Unsupported dispositions.
   if (disposition == NEW_BACKGROUND_TAB || disposition == NEW_WINDOW ||
       disposition == NEW_FOREGROUND_TAB)
     return false;
 
   // The omnibox currently doesn't use other dispositions, so we don't attempt
   // to handle them. If you hit this DCHECK file a bug and I'll (sky) add
   // support for the new disposition.
   DCHECK(disposition == CURRENT_TAB) << disposition;
 
-  // If we will not be replacing search terms from this URL, don't send to
-  // InstantController.
+  Profile* browser_profile = profile();
   const base::string16& search_terms =
-      chrome::GetSearchTermsFromURL(browser_->profile(), url);
+      chrome::ExtractSearchTermsFromURL(browser_profile, url);
   if (search_terms.empty())
     return false;
 
   InstantSearchPrerenderer* prerenderer =
-      GetInstantSearchPrerenderer(profile());
+      GetInstantSearchPrerenderer(browser_profile);
   if (prerenderer) {
     if (prerenderer->CanCommitQuery(GetActiveWebContents(), search_terms)) {
       // Submit query to render the prefetched results. Browser will swap the
       // prerendered contents with the active tab contents.
       prerenderer->Commit(search_terms);
       return false;
     } else {
       prerenderer->Cancel();
     }
   }
 
-  return instant_.SubmitQuery(search_terms);
+  // If we will not be replacing search terms from this URL, don't send to
+  // InstantController.
+  return chrome::IsSuitableURLForInstant(browser_profile, url) ?

[Jered, 2014/02/11 19:10:10]

nit:
if (!chrome::IsSuitableURLForInstant(browser_profile, url))
  return false;

return instant_.SubmitQuery(search_terms);

[kmadhusu, 2014/02/11 23:24:32]

Done.

+      instant_.SubmitQuery(search_terms) : false;
 }
 
 Profile* BrowserInstantController::profile() const {
   return browser_->profile();
 }
 
 content::WebContents* BrowserInstantController::GetActiveWebContents() const {
   return browser_->tab_strip_model()->GetActiveWebContents();
 }
 
@@ skipping 62 matching lines @@
     content::RenderProcessHost* rph = contents->GetRenderProcessHost();
     instant_service->SendSearchURLsToRenderer(rph);
 
     // Reload the contents to ensure that it gets assigned to a non-priviledged
     // renderer.
     if (!instant_service->IsInstantProcess(rph->GetID()))
       continue;
     contents->GetController().Reload(false);
   }
 }