Enable query extraction for Android

chrome/browser/search_engines/search_terms_data.cc

Index: chrome/browser/search_engines/search_terms_data.cc
diff --git a/chrome/browser/search_engines/search_terms_data.cc b/chrome/browser/search_engines/search_terms_data.cc
index a1adfc9f4c8ae633b65957e100c7a54fb1dd74a4..1b15d35b1d80738222991286aed395ea6ff9cd03 100644
--- a/chrome/browser/search_engines/search_terms_data.cc
+++ b/chrome/browser/search_engines/search_terms_data.cc
@@ -141,6 +141,12 @@ std::string UIThreadSearchTermsData::InstantEnabledParam() const {
 }
 
 std::string UIThreadSearchTermsData::InstantExtendedEnabledParam() const {
+#if defined(OS_IOS) || defined(OS_ANDROID)

[Peter Kasting, 2013/03/01 21:44:44]

You're omitting the threading assertion and the profile_ NULL-check, both of
which seem necessary.

[Yusuf, 2013/03/01 22:26:07]

Done.

+  if (chrome::search::IsQueryExtractionEnabled(profile_) &&
+      GURL(GoogleBaseURLValue()).SchemeIsSecure())

[Peter Kasting, 2013/03/01 21:44:44]

Shouldn't we be checking scheme security elsewhere?  Instant extended certainly
doesn't check it here.

[Yusuf, 2013/03/01 22:26:07]

You are right. Apparently it is already checked in getSearchTerms which we will
be using, so removing this from here.

On 2013/03/01 21:44:44, Peter Kasting wrote:

[rohitrao (ping after 24h), 2013/03/01 22:38:26]

We have to be a little careful here, because we don't want to append espv=1 for
HTTP urls.  While getSearchTerms() will check for https before performing terms
extraction, if we blindly add espv=1, then the server might incorrectly return
the 1993 ui for HTTP searches.

+    return std::string(google_util::kInstantExtendedAPIParam) + "=" +
+        std::string(google_util::kInstantExtendedDefaultAPIVersion) + "&";

[Peter Kasting, 2013/03/01 21:44:44]

If you're already modifying
UIThreadSearchTermsData::InstantExtendedEnabledParam() as your way of
accomplishing query extraction, why not modify
chrome::search::EmbeddedSearchPageVersion() to return the version number you
want?  Then you could avoid the #ifs here and, if I'm not mistaken, in the other
places you're touching as well.

[Yusuf, 2013/03/01 22:26:07]

Done.

+#else
   DCHECK(!BrowserThread::IsWellKnownThread(BrowserThread::UI) ||
          BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (profile_) {
@@ -151,6 +157,7 @@ std::string UIThreadSearchTermsData::InstantExtendedEnabledParam() const {
           base::Uint64ToString(instant_extended_api_version) + "&";
     }
   }
+#endif
   return std::string();
 }