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

Unified Diff: content/public/browser/render_frame_host.h

Issue 2857263003: Task Manager should listen to WebContentsObserver::RenderFrameCreated. (Closed)
Patch Set: Tweaked the comments as suggested in the CR feedback Created 3 years, 7 months 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/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 72cbc18bbafaea6d07ae5e29f5e9c42fc69c5d0a..5b33b551add3ebcd8d4971f046b663fa3eaf3eaf 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -198,6 +198,16 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
// and still has a connection. This is valid for all frames.
virtual bool IsRenderFrameLive() = 0;
+ // Returns true if this is the currently-visible RenderFrameHost for our frame
+ // tree node. During process transfer, a RenderFrameHost may be created that
+ // is not current. After process transfer, the old RenderFrameHost becomes
+ // non-current until it is deleted (which may not happen until its unload
+ // handler runs).
+ //
+ // Changes to the IsCurrent() state of a RenderFrameHost may be observed via
+ // WebContentsObserver::RenderFrameHostChanged().
+ virtual bool IsCurrent() = 0;
+
// Get the number of proxies to this frame, in all processes. Exposed for
// use by resource metrics.
virtual int GetProxyCount() = 0;
« no previous file with comments | « content/browser/renderer_host/render_view_host_impl.cc ('k') | content/public/browser/render_process_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698