While loading url in current tab should check if any prerender page avaliable.

chrome/browser/android/tab_android.cc

 // Copyright (c) 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/android/tab_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/debug/trace_event.h"
@@ skipping 61 matching lines @@
       create_params.opener = params.source_contents;
   }
   if (params.disposition == NEW_BACKGROUND_TAB)
     create_params.initially_hidden = true;
 
   WebContents* target_contents = WebContents::Create(create_params);
 
   return target_contents;
 }
 
+bool SwapWithPrerender(const GURL& url, chrome::NavigateParams* params) {

[Bernhard Bauer, 2014/07/28 15:00:05]

MaybeSwapWithPrerender()?

[Jitu( very slow this week), 2014/07/30 13:48:28]

using only this function MaybeSwapWithPrerender() is fine here.
No need to to use UsePrerenderedPage() as this doing the same thing but just
updating the target_contents.

Which we have taken care in HandlePopupNavigation()

[Bernhard Bauer, 2014/07/30 14:19:46]

What I was suggesting was to name this method MaybeSwapWithPrerender(), as it is
not guaranteed to swap with a prerendered page.

+  Profile* profile =
+      Profile::FromBrowserContext(params->target_contents->GetBrowserContext());
+  InstantSearchPrerenderer* prerenderer =
+      InstantSearchPrerenderer::GetForProfile(profile);
+  if (prerenderer && prerenderer->UsePrerenderedPage(url, params))

[Bernhard Bauer, 2014/07/28 15:00:05]

When is |prerenderer| false?

[Jitu( very slow this week), 2014/07/30 13:48:27]

Removed this as not required.

Thanks.

+    return true;
+
+  prerender::PrerenderManager* prerender_manager =
+      prerender::PrerenderManagerFactory::GetForProfile(profile);
+  return prerender_manager &&

[Bernhard Bauer, 2014/07/28 15:00:05]

When is |prerender_manager| false?

[Jitu( very slow this week), 2014/07/30 13:48:28]

Changed.

Thanks

[Bernhard Bauer, 2014/07/30 14:19:46]

If |prerender_manager| can be NULL, the original code is fine, but you didn't
answer my question. Under what circumstances is this null? It would be good to
explain those circumstances in a comment. If it actually can't be NULL, you can
remove the whole check.

+         prerender_manager->MaybeUsePrerenderedPage(url, params);
+}
+
 }  // namespace
 
 TabAndroid* TabAndroid::FromWebContents(content::WebContents* web_contents) {
   CoreTabHelper* core_tab_helper = CoreTabHelper::FromWebContents(web_contents);
   if (!core_tab_helper)
     return NULL;
 
   CoreTabHelperDelegate* core_delegate = core_tab_helper->delegate();
   if (!core_delegate)
     return NULL;
@@ skipping 88 matching lines @@
   Java_Tab_setSyncId(env, weak_java_tab_.get(env).obj(), sync_id);
 }
 
 void TabAndroid::HandlePopupNavigation(chrome::NavigateParams* params) {
   if (params->disposition != SUPPRESS_OPEN &&
       params->disposition != SAVE_TO_DISK &&
       params->disposition != IGNORE_ACTION) {
     if (!params->url.is_empty()) {
       bool was_blocked = false;
       GURL url(params->url);
-      NavigationController::LoadURLParams load_url_params(url);
-      MakeLoadURLParams(params, &load_url_params);
       if (params->disposition == CURRENT_TAB) {
-        web_contents_.get()->GetController().LoadURLWithParams(load_url_params);
+        bool swapped_with_prerender = false;
+        params->target_contents = web_contents_.get();
+        swapped_with_prerender = SwapWithPrerender(url, params);
+        if (!swapped_with_prerender) {

[Bernhard Bauer, 2014/07/28 15:00:05]

You can inline this check.

[Jitu( very slow this week), 2014/07/30 13:48:28]

Done.

Thanks

+          NavigationController::LoadURLParams load_url_params(url);
+          MakeLoadURLParams(params, &load_url_params);
+          params->target_contents->GetController().LoadURLWithParams(
+              load_url_params);
+        }
       } else {
         params->target_contents = CreateTargetContents(*params, url);
+        NavigationController::LoadURLParams load_url_params(url);
+        MakeLoadURLParams(params, &load_url_params);
         params->target_contents->GetController().LoadURLWithParams(
             load_url_params);
         web_contents_delegate_->AddNewContents(params->source_contents,
                                                params->target_contents,
                                                params->disposition,
                                                params->window_bounds,
                                                params->user_gesture,
                                                &was_blocked);
         if (was_blocked)
           params->target_contents = NULL;
@@ skipping 375 matching lines @@
 
 static void Init(JNIEnv* env, jobject obj) {
   TRACE_EVENT0("native", "TabAndroid::Init");
   // This will automatically bind to the Java object and pass ownership there.
   new TabAndroid(env, obj);
 }
 
 bool TabAndroid::RegisterTabAndroid(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }