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

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
@@ skipping 1542 matching lines @@
 
 void WebContentsImpl::ResizeDueToAutoResize(const gfx::Size& new_size) {
   if (delegate_)
     delegate_->ResizeDueToAutoResize(this, new_size);
 }
 
 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_,
                     DidOpenURL(params.url, params.referrer,
                                params.disposition, params.transition));
   return new_contents;
 }
 
 bool WebContentsImpl::Send(IPC::Message* message) {
   if (!GetRenderViewHost()) {
@@ skipping 1844 matching lines @@
 
 BrowserPluginGuest* WebContentsImpl::GetBrowserPluginGuest() {
   return browser_plugin_guest_.get();
 }
 
 BrowserPluginEmbedder* WebContentsImpl::GetBrowserPluginEmbedder() {
   return browser_plugin_embedder_.get();
 }
 
 }  // namespace content