Add ctxsl_alternate_term to the contextual search request and update the API version to 2.

chrome/browser/search_engines/template_url_service_android.cc

Index: chrome/browser/search_engines/template_url_service_android.cc
diff --git a/chrome/browser/search_engines/template_url_service_android.cc b/chrome/browser/search_engines/template_url_service_android.cc
index a759cf3d4d4f66b7909bda4e0a19d1f2d11fd974..ac2d535b69af7b7ab92fda5793836d27001f602c 100644
--- a/chrome/browser/search_engines/template_url_service_android.cc
+++ b/chrome/browser/search_engines/template_url_service_android.cc
@@ -20,6 +20,7 @@
 #include "net/base/url_util.h"
 
 using base::android::ConvertJavaStringToUTF16;
+using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF16ToJavaString;
 using base::android::ConvertUTF8ToJavaString;

[Peter Kasting, 2014/08/13 02:04:16]

Nit: Avoid using statements unless they significantly improve code readability
(I'd remove all these)

[jeremycho, 2014/08/13 20:29:27]

Done.

 
@@ -204,15 +205,30 @@ base::android::ScopedJavaLocalRef<jstring>
 TemplateUrlServiceAndroid::GetUrlForContextualSearchQuery(JNIEnv* env,
                                                           jobject obj,
                                                           jstring jquery) {
+  return GetUrlForContextualSearchQuery(env, obj, jquery, NULL);
+}
+
+base::android::ScopedJavaLocalRef<jstring>
+TemplateUrlServiceAndroid::GetUrlForContextualSearchQuery(
+    JNIEnv* env, jobject obj, jstring jquery, jstring jalternate_term) {

[Peter Kasting, 2014/08/13 02:04:16]

Nit: One arg per line

[jeremycho, 2014/08/13 20:29:27]

Done.

   base::string16 query(ConvertJavaStringToUTF16(env, jquery));
   std::string url;
 
   if (!query.empty()) {
     GURL gurl(GetDefaultSearchURLForSearchTerms(template_url_service_, query));
     if (google_util::IsGoogleSearchUrl(gurl)) {
-      gurl = net::AppendQueryParameter(gurl, "ctxs", "1");
+      gurl = net::AppendQueryParameter(gurl, "ctxs", "2");
       // Indicate that the search page is being prefetched.
       gurl = net::AppendQueryParameter(gurl, "pf", "c");
+
+      if (jalternate_term) {
+        std::string alternate_term(
+            ConvertJavaStringToUTF8(env, jalternate_term));

[Peter Kasting, 2014/08/13 02:04:15]

Nit: Is it safe to run this if !jalternate_term?  Because if so it seems like
the conditional below would catch that case.

[jeremycho, 2014/08/13 20:29:27]

A null is being passed from Java, so I think having the check is good to avoid a
warning from ConvertJavaStringToUTF8.

+        if (!alternate_term.empty()) {
+          gurl = net::AppendQueryParameter(
+              gurl, "ctxsl_alternate_term", alternate_term);
+        }
+      }
     }
     url = gurl.spec();
   }