Commit instant loader when the instant page navigates away from instant URL.

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 709debdb4ac5f52ecc2bc1b260e785295ec4529a..270c15f82997c1051fdb3fe55c34ab3972db56e0 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1324,6 +1324,11 @@ void Browser::MaybeUpdateBookmarkBarStateForInstantPreview(
 
 WebContents* Browser::OpenURLFromTab(WebContents* source,
                                      const OpenURLParams& params) {
+  // Notify instant about the navigation. This needs to happen before so that
+  // instant can commit if required.
+  if (instant_controller_)

[sky, 2012/12/07 22:01:46]

Does this really do what you want? Seems like you want
WebContentsObserver::NavigateToPendingEntry

[Shishir, 2012/12/08 00:14:54]

Ok I got this  wrong. The change does not belong here but in the
InstantLoader::WebContentsDelegate which is the delegate for the InstantPage and
not the browser. I think this change should make it clear why we need to make
the change in OpenURLFromTab. 

+    instant_controller_->AboutToOpenURL(source, params.url);
+
   chrome::NavigateParams nav_params(this, params.url, params.transition);
   nav_params.source_contents = source;
   nav_params.referrer = params.referrer;