Prerender/Instant interopability.

chrome/browser/instant/instant_loader.cc

 // Copyright (c) 2011 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/instant/instant_loader.h"
 
 #include <algorithm>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 121 matching lines @@
       host->Send(new ViewMsg_DetermineIfPageSupportsInstant(
           host->routing_id(), text_, verbatim_, text_length, text_length));
       break;
     }
     default:
       NOTREACHED();
       break;
   }
 }
 
+// If this is being called, something is swapping in to our preview_contents_
+// before we've added it to the tab strip.
+void InstantLoader::SwapTabContents(TabContentsWrapper* old_tc,
+                                    TabContentsWrapper* new_tc) {
+  DCHECK(old_tc == preview_contents_);
+  // We release here without deleting so that the caller still has reponsibility
+  // for deleting the TabContentsWrapper. The (void) cast is so that we use the
+  // returned value.
+  TabContentsWrapper* tab_contents_wrapper = preview_contents_.release();

[sky, 2011/05/19 19:53:55]

ignore_result

[dominich, 2011/05/19 23:57:28]

Done.

+  (void) tab_contents_wrapper;
+  preview_contents_.reset(new_tc);

[sky, 2011/05/19 19:53:55]

I'm not sure under when this might be invoked. Couple of things that likely need
to be done:

. See the code in CreatePreviewContents. I think we need to do all that on the
tabcontents. Be sure to remove TabContentsDelegateImpl as it is still installed
on the old delegate.
. I assume if this happens the page is ready to be shown. In which case invoke
ShowPreview.
. CreatePreviewContents makes sure to adjust the page ids. If swap is called
what happens to page id of the new tabcontents? Are we sure the id is >= id of
old?

[dominich, 2011/05/19 23:57:28]

It's invoked if the InstantLoader is the delegate for a TabContentsWrapper that
is being swapped for a Prerender TabContents.

This is called as a side-effect of the ProvisionalChangeToMainFrameUrl in the
TabContentsObserver so shouldn't disturb the rest of the Instant flow.
Done
This shouldn't be necessary as we're about to go through the Instant commit flow
as normal.
While the Prerender system has already increased the page id based on the source
of the prerender, it can't hurt to make sure we're increasing it here too.

+}
+
 // TabContentsDelegateImpl -----------------------------------------------------
 
 class InstantLoader::TabContentsDelegateImpl
     : public TabContentsDelegate,
       public NotificationObserver,
       public TabContentsObserver,
       public DownloadTabHelperDelegate {
  public:
   explicit TabContentsDelegateImpl(InstantLoader* loader);
 
@@ skipping 837 matching lines @@
     host->Send(new ViewMsg_SearchBoxResize(
         host->routing_id(), GetOmniboxBoundsInTermsOfPreview()));
   }
 }
 
 void InstantLoader::CreatePreviewContents(TabContentsWrapper* tab_contents) {
   TabContents* new_contents =
       new TabContents(
           tab_contents->profile(), NULL, MSG_ROUTING_NONE, NULL, NULL);
   preview_contents_.reset(new TabContentsWrapper(new_contents));
+  preview_contents_->set_delegate(this);
   preview_contents_->blocked_content_tab_helper()->SetAllContentsBlocked(true);
   // Propagate the max page id. That way if we end up merging the two
   // NavigationControllers (which happens if we commit) none of the page ids
   // will overlap.
   int32 max_page_id = tab_contents->tab_contents()->GetMaxPageID();
   if (max_page_id != -1)
     preview_contents_->controller().set_max_restored_page_id(max_page_id + 1);
 
   preview_tab_contents_delegate_.reset(new TabContentsDelegateImpl(this));
   new_contents->set_delegate(preview_tab_contents_delegate_.get());
@@ skipping 18 matching lines @@
       Source<NavigationController>(&preview_contents_->controller()));
 #endif
 
   registrar_.Add(
       this,
       NotificationType::NAV_ENTRY_COMMITTED,
       Source<NavigationController>(&preview_contents_->controller()));
 
   preview_contents_->tab_contents()->ShowContents();
 }