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

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index e4e4e6d7ccd0b3881dd0b5398fbb1d93ae68bea5..785e48544ca8501b5e09f6bdc458abbee5250113 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1560,6 +1560,9 @@ WebContents* WebContentsImpl::OpenURL(const OpenURLParams& params) {
   if (!delegate_)
     return NULL;
 
+  FOR_EACH_OBSERVER(WebContentsObserver, observers_,
+                    AboutToOpenURL(params.url));

[jam, 2012/12/07 06:18:15]

who's the WebContentsDelegate that gets called right below this? is it Browser?
i.e. can you get that one to call the instant code so that the content api
doesn't need to get another callback?

[Shishir, 2012/12/07 21:01:25]

We would like to get this call before the new WebContents is created and the
navigation happens so that we can commit the instant page. After the new
WebContents is created, committing the instant page will lead to the tab having
the wrong Webcontents.

[jam, 2012/12/07 21:21:02]

maybe I wasn't clear. I'm not talking about changing the order of when you get
notified. I'm asking if you can get this notification by code change in
src/chrome alone, without modifying WebContentsObserver. delegate_ is
implemented in chrome.

+
   WebContents* new_contents = delegate_->OpenURLFromTab(this, params);
   // Notify observers.
   FOR_EACH_OBSERVER(WebContentsObserver, observers_,