Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(695)

Unified Diff: content/browser/web_contents/web_contents_impl.h

Issue 104833006: Switch ContentSettingsObserver to be a RenderFrameObserver instead of a RenderViewObserver (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.h
===================================================================
--- content/browser/web_contents/web_contents_impl.h (revision 240912)
+++ content/browser/web_contents/web_contents_impl.h (working copy)
@@ -174,6 +174,9 @@
virtual BrowserContext* GetBrowserContext() const OVERRIDE;
virtual RenderProcessHost* GetRenderProcessHost() const OVERRIDE;
virtual RenderFrameHost* GetMainFrame() OVERRIDE;
+ virtual void ForEachFrame(
+ const base::Callback<void(RenderFrameHost*)>& on_frame) OVERRIDE;
+ virtual void SendToAllFrames(IPC::Message* message) OVERRIDE;
virtual RenderViewHost* GetRenderViewHost() const OVERRIDE;
virtual void GetRenderViewHostAtPosition(
int x,
@@ -284,6 +287,7 @@
const IPC::Message& message) OVERRIDE;
virtual void RenderFrameCreated(RenderFrameHost* render_frame_host) OVERRIDE;
virtual void RenderFrameDeleted(RenderFrameHost* render_frame_host) OVERRIDE;
+ virtual WebContents* GetAsWebContents() OVERRIDE;
// RenderViewHostDelegate ----------------------------------------------------
virtual RenderViewHostDelegateView* GetDelegateView() OVERRIDE;
@@ -294,7 +298,9 @@
virtual const GURL& GetURL() const OVERRIDE;
virtual const GURL& GetVisibleURL() const OVERRIDE;
virtual const GURL& GetLastCommittedURL() const OVERRIDE;
- virtual WebContents* GetAsWebContents() OVERRIDE;
+ // RenderFrameHostDelegate has the same method, so list it there because this
+ // interface is going away.
+ // virtual WebContents* GetAsWebContents() OVERRIDE;
virtual gfx::Rect GetRootWindowResizerRect() const OVERRIDE;
virtual void RenderViewCreated(RenderViewHost* render_view_host) OVERRIDE;
virtual void RenderViewReady(RenderViewHost* render_view_host) OVERRIDE;
@@ -544,9 +550,9 @@
int merge_history_length,
int32 minimum_page_id) OVERRIDE;
- // Called by InterstitialPageImpl when it creates a RenderViewHost.
- virtual void RenderViewForInterstitialPageCreated(
- RenderViewHost* render_view_host) OVERRIDE;
+ // Called by InterstitialPageImpl when it creates a RenderFrameHost.
+ virtual void RenderFrameForInterstitialPageCreated(
+ RenderFrameHost* render_frame_host) OVERRIDE;
// Sets the passed interstitial as the currently showing interstitial.
// No interstitial page should already be attached.

Powered by Google App Engine
This is Rietveld 408576698