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

Unified Diff: content/browser/frame_host/render_frame_host_manager.cc

Issue 1981413002: Remove Is{RVH}OnSwappedOutList, now that swappedout:// is gone. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update comments Created 4 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/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index 2b8794e052f62f6ae570f704ad6d84722e3bf149..af969a4a6e8220771f60442cd0f067a39a82bb20 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -2304,30 +2304,6 @@ std::unique_ptr<RenderFrameHostImpl> RenderFrameHostManager::SetRenderFrameHost(
return old_render_frame_host;
}
-bool RenderFrameHostManager::IsRVHOnSwappedOutList(
- RenderViewHostImpl* rvh) const {
- RenderFrameProxyHost* proxy = GetRenderFrameProxyHost(rvh->GetSiteInstance());
- if (!proxy)
- return false;
- // If there is a proxy without RFH, it is for a subframe in the SiteInstance
- // of |rvh|. Subframes should be ignored in this case.
- if (!proxy->render_frame_host())
- return false;
- return IsOnSwappedOutList(proxy->render_frame_host());
-}
-
-bool RenderFrameHostManager::IsOnSwappedOutList(
- RenderFrameHostImpl* rfh) const {
- if (!rfh->GetSiteInstance())
- return false;
-
- RenderFrameProxyHost* host = GetRenderFrameProxyHost(rfh->GetSiteInstance());
- if (!host)
- return false;
-
- return host->render_frame_host() == rfh;
-}
-
RenderViewHostImpl* RenderFrameHostManager::GetSwappedOutRenderViewHost(
SiteInstance* instance) const {
RenderFrameProxyHost* proxy = GetRenderFrameProxyHost(instance);

Powered by Google App Engine
This is Rietveld 408576698