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

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

Issue 92153003: Rename RenderViewHostManager to RenderFrameHostManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 1 month 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.h
diff --git a/content/browser/frame_host/render_view_host_manager.h b/content/browser/frame_host/render_frame_host_manager.h
similarity index 96%
rename from content/browser/frame_host/render_view_host_manager.h
rename to content/browser/frame_host/render_frame_host_manager.h
index 880958926bba8a5c9edb01e1aa04964b3d524d7c..78ca04aefef436cf132e07b325f5326befe84fe9 100644
--- a/content/browser/frame_host/render_view_host_manager.h
+++ b/content/browser/frame_host/render_frame_host_manager.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_VIEW_HOST_MANAGER_H_
-#define CONTENT_BROWSER_FRAME_HOST_RENDER_VIEW_HOST_MANAGER_H_
+#ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
+#define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
#include "base/basictypes.h"
#include "base/logging.h"
@@ -23,18 +23,17 @@ class InterstitialPageImpl;
class NavigationControllerImpl;
class NavigationEntry;
class NavigationEntryImpl;
+class RenderFrameHostManagerTest;
class RenderViewHost;
class RenderViewHostImpl;
-class RenderViewHostManagerTest;
class RenderWidgetHostDelegate;
class RenderWidgetHostView;
class TestWebContents;
class WebUIImpl;
-// Manages RenderViewHosts for a WebContentsImpl. Normally there is only one and
-// it is easy to do. But we can also have transitions of processes (and hence
-// RenderViewHosts) that can get complex.
-class CONTENT_EXPORT RenderViewHostManager
+// Manages RenderFrameHosts for a FrameTreeNode. This class acts as a state
+// machine to make cross-process navigations in a frame possible.
+class CONTENT_EXPORT RenderFrameHostManager
: public RenderViewHostDelegate::RendererManagement,
public NotificationObserver {
public:
@@ -113,11 +112,11 @@ class CONTENT_EXPORT RenderViewHostManager
// RenderViewHosts that are created.
//
// You must call Init() before using this class.
- RenderViewHostManager(
+ RenderFrameHostManager(
RenderViewHostDelegate* render_view_delegate,
RenderWidgetHostDelegate* render_widget_delegate,
Delegate* delegate);
- virtual ~RenderViewHostManager();
+ virtual ~RenderFrameHostManager();
// For arguments, see WebContentsImpl constructor.
void Init(BrowserContext* browser_context,
@@ -248,7 +247,7 @@ class CONTENT_EXPORT RenderViewHostManager
void SwapOutOldPage();
private:
- friend class RenderViewHostManagerTest;
+ friend class RenderFrameHostManagerTest;
friend class TestWebContents;
// Tracks information about a navigation while a cross-process transition is
@@ -399,9 +398,9 @@ class CONTENT_EXPORT RenderViewHostManager
NotificationRegistrar registrar_;
- DISALLOW_COPY_AND_ASSIGN(RenderViewHostManager);
+ DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
};
} // namespace content
-#endif // CONTENT_BROWSER_FRAME_HOST_RENDER_VIEW_HOST_MANAGER_H_
+#endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
« no previous file with comments | « content/browser/frame_host/frame_tree_node.cc ('k') | content/browser/frame_host/render_frame_host_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698