Add support to enable EmbeddedSearch API in Android Chrome search results page.

chrome/browser/android/tab_android.cc

Index: chrome/browser/android/tab_android.cc
diff --git a/chrome/browser/android/tab_android.cc b/chrome/browser/android/tab_android.cc
index 773d1f047f9eb4250ad17419dfef609ff0b9519b..a362bf2a3c2a0d1334a986d94f09d0bc6a929b37 100644
--- a/chrome/browser/android/tab_android.cc
+++ b/chrome/browser/android/tab_android.cc
@@ -21,6 +21,7 @@
 #include "chrome/browser/printing/print_view_manager_basic.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_android.h"
+#include "chrome/browser/search/search.h"
 #include "chrome/browser/sessions/session_tab_helper.h"
 #include "chrome/browser/sync/glue/synced_tab_delegate_android.h"
 #include "chrome/browser/ui/android/content_settings/popup_blocked_infobar_delegate.h"
@@ -30,6 +31,7 @@
 #include "chrome/browser/ui/android/tab_model/tab_model_list.h"
 #include "chrome/browser/ui/android/window_android_helper.h"
 #include "chrome/browser/ui/blocked_content/popup_blocker_tab_helper.h"
+#include "chrome/browser/ui/search/search_tab_helper.h"
 #include "chrome/browser/ui/tab_contents/core_tab_helper.h"
 #include "chrome/browser/ui/tab_helpers.h"
 #include "chrome/browser/ui/toolbar/toolbar_model_impl.h"
@@ -461,6 +463,15 @@ TabAndroid::TabLoadStatus TabAndroid::LoadUrl(JNIEnv* env,
           GURL(base::android::ConvertJavaStringToUTF8(env, j_referrer_url)),
           static_cast<blink::WebReferrerPolicy>(referrer_policy));
     }
+    const base::string16 search_terms =
+        chrome::ExtractSearchTermsFromURL(GetProfile(), gurl);
+    SearchTabHelper* search_tab_helper =
+        SearchTabHelper::FromWebContents(web_contents_.get());
+    if (!search_terms.empty() && search_tab_helper &&
+        search_tab_helper->SupportsInstant()) {
+      search_tab_helper->Submit(search_terms);
+      return DEFAULT_PAGE_LOAD;

[samarth, 2014/04/25 06:50:20]

Is this the right return type here?

[kmadhusu, 2014/04/25 17:23:19]

Ted@, Is this okay? Do you want me to create a new status?

[Ted C, 2014/04/29 16:53:13]

This should be fine.  This isn't a prerendered page, so I think this is the
"most" correct.  We might want to add a new type at some point if we need to
differentiate, but until we do this is ok.

The one thing this made me think of is whether the loading progress bar is shown
when going through this path?  We used these load value types to simulate load
progress in the past, so if this isn't happening for these pages then we might
want to add a new type to handle that.

[kmadhusu, 2014/04/29 17:01:30]

Loading progress bar is shown while going through this path. Looks like it works
as expected.

+    }
     content_view->LoadUrl(load_params);
   }
   return DEFAULT_PAGE_LOAD;