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

Unified Diff: chrome/browser/renderer_host/chrome_render_view_host_observer.h

Issue 7612016: Tie extension/app initialization to RenderView creation, not RenderViewHost creation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comment.x Created 9 years, 4 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: chrome/browser/renderer_host/chrome_render_view_host_observer.h
diff --git a/chrome/browser/renderer_host/chrome_render_view_host_observer.h b/chrome/browser/renderer_host/chrome_render_view_host_observer.h
index c5469c489628bc8e68aa3bdf7927f0b5604af743..411a953bcfd2561e79f79da03382d4c25ca8deed 100644
--- a/chrome/browser/renderer_host/chrome_render_view_host_observer.h
+++ b/chrome/browser/renderer_host/chrome_render_view_host_observer.h
@@ -8,6 +8,8 @@
#include "content/browser/renderer_host/render_view_host_observer.h"
+class Extension;
+
// This class holds the Chrome specific parts of RenderViewHost, and has the
// same lifetime.
class ChromeRenderViewHostObserver : public RenderViewHostObserver {
@@ -16,10 +18,20 @@ class ChromeRenderViewHostObserver : public RenderViewHostObserver {
virtual ~ChromeRenderViewHostObserver();
// RenderViewHostObserver overrides.
+ virtual void RenderViewHostInitialized() OVERRIDE;
virtual void Navigate(const ViewMsg_Navigate_Params& params) OVERRIDE;
virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
private:
+ // Does extension-specific initialization when a new RenderViewHost is
+ // created.
+ void InitRenderViewHostForExtensions();
+ // Does extension-specific initialization when a new renderer process is
+ // created by a RenderViewHost.
+ void InitRenderViewForExtensions();
+ // Gets the extension or app (if any) that is associated with the RVH.
+ const Extension* GetExtension();
+
void OnDomOperationResponse(const std::string& json_string,
int automation_id);
« no previous file with comments | « chrome/browser/extensions/isolated_app_apitest.cc ('k') | chrome/browser/renderer_host/chrome_render_view_host_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698