Only delete old frames when a new main frame navigation commits

chrome/browser/extensions/api/web_navigation/web_navigation_api.h

 // 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.
 
 // Defines the Chrome Extensions WebNavigation API functions for observing and
 // intercepting navigation events, as specified in the extension JSON API.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_WEB_NAVIGATION_WEB_NAVIGATION_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_WEB_NAVIGATION_WEB_NAVIGATION_API_H_
 
@@ skipping 22 matching lines @@
   explicit WebNavigationTabObserver(content::WebContents* web_contents);
   virtual ~WebNavigationTabObserver();
 
   // Returns the object for the given |tab_contents|.
   static WebNavigationTabObserver* Get(content::WebContents* web_contents);
 
   const FrameNavigationState& frame_navigation_state() const {
     return navigation_state_;
   }
 
+  content::RenderViewHost* GetRenderViewHostInProcess(int process_id) const;
+
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
 
   // content::WebContentsObserver implementation.
   virtual void AboutToNavigateRenderView(
       content::RenderViewHost* render_view_host) OVERRIDE;
   virtual void DidStartProvisionalLoadForFrame(
@@ skipping 37 matching lines @@
                                    content::PageTransition transition,
                                    int64 source_frame_num) OVERRIDE;
   virtual void WebContentsDestroyed(content::WebContents* tab) OVERRIDE;
 
  private:
   // True if the transition and target url correspond to a reference fragment
   // navigation.
   bool IsReferenceFragmentNavigation(FrameNavigationState::FrameID frame_id,
                                      const GURL& url);
 
+  // Creates and sends onErrorOccurred events for all on-going navigations. If
+  // |render_view_host| is non-NULL, only generates events for frames in this
+  // render view host.
+  void SendErrorEvents(content::WebContents* web_contents,
+                       content::RenderViewHost* render_view_host);
+
   // Tracks the state of the frames we are sending events for.
   FrameNavigationState navigation_state_;
 
   // Used for tracking registrations to redirect notifications.
   content::NotificationRegistrar registrar_;
 
   content::RenderViewHost* render_view_host_;
   content::RenderViewHost* pending_render_view_host_;
 
   DISALLOW_COPY_AND_ASSIGN(WebNavigationTabObserver);
@@ skipping 81 matching lines @@
 // API function that returns the states of all frames in a given tab.
 class GetAllFramesFunction : public SyncExtensionFunction {
   virtual ~GetAllFramesFunction() {}
   virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME("webNavigation.getAllFrames")
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_WEB_NAVIGATION_WEB_NAVIGATION_API_H_