Expose getSearchTerms to tab_model for Android

chrome/browser/ui/search/search.cc

Index: chrome/browser/ui/search/search.cc
diff --git a/chrome/browser/ui/search/search.cc b/chrome/browser/ui/search/search.cc
index e6d79bd0b396f29bd7fe61063768d7ef381625c2..48fc2aa3776e55667b17a4a805736cef658ac906 100644
--- a/chrome/browser/ui/search/search.cc
+++ b/chrome/browser/ui/search/search.cc
@@ -192,7 +192,7 @@ string16 GetSearchTerms(const content::WebContents* contents) {
   Profile* profile = Profile::FromBrowserContext(contents->GetBrowserContext());
   if (!IsQueryExtractionEnabled(profile))
     return string16();
-
+#if !defined(OS_IOS) && !defined(OS_ANDROID)

[Ted C, 2013/03/01 02:50:21]

Should we add a comment that IOS and Android do not support instant and that is
why we skip this check?  Or is that just implied knowledge?

[Yusuf, 2013/03/01 18:58:20]

Added comment.

On 2013/03/01 02:50:21, Ted C wrote:

   // For security reasons, don't extract search terms if the page is not being
   // rendered in the privileged Instant renderer process. This is to protect
   // against a malicious page somehow scripting the search results page and
@@ -211,7 +211,7 @@ string16 GetSearchTerms(const content::WebContents* contents) {
 
   if (!instant_service->IsInstantProcess(process_host->GetID()))
     return string16();
-
+#endif

[dhollowa, 2013/03/01 00:35:51]

nit:  // !defined(OS_IOS) && !defined(OS_ANDROID)

[Yusuf, 2013/03/01 18:58:20]

Done.

   // Check to see if search terms have already been extracted.
   const content::NavigationEntry* entry =
       contents->GetController().GetVisibleEntry();