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

Side by Side Diff: content/browser/content_browser_client.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: Switch to initializing extensions when a RenderView is created, as opposed to a RenderViewHost. 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_ 5 #ifndef CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_
6 #define CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_ 6 #define CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 // thread unless otherwise specified. Use this "escape hatch" sparingly, to 61 // thread unless otherwise specified. Use this "escape hatch" sparingly, to
62 // avoid the embedder interface ballooning and becoming very specific to Chrome. 62 // avoid the embedder interface ballooning and becoming very specific to Chrome.
63 // (Often, the call out to the client can happen in a different part of the code 63 // (Often, the call out to the client can happen in a different part of the code
64 // that either already has a hook out to the embedder, or calls out to one of 64 // that either already has a hook out to the embedder, or calls out to one of
65 // the observer interfaces.) 65 // the observer interfaces.)
66 class ContentBrowserClient { 66 class ContentBrowserClient {
67 public: 67 public:
68 // Notifies that a new RenderHostView has been created. 68 // Notifies that a new RenderHostView has been created.
69 virtual void RenderViewHostCreated(RenderViewHost* render_view_host) = 0; 69 virtual void RenderViewHostCreated(RenderViewHost* render_view_host) = 0;
70 70
71 // Notifies that a new RenderView has been created.
Charlie Reis 2011/08/11 16:53:43 Nit: "for the given RenderViewHost."
72 virtual void RenderViewCreated(RenderViewHost* render_view_host) = 0;
73
71 // Notifies that a BrowserRenderProcessHost has been created. This is called 74 // Notifies that a BrowserRenderProcessHost has been created. This is called
72 // before the content layer adds its own BrowserMessageFilters, so that the 75 // before the content layer adds its own BrowserMessageFilters, so that the
73 // embedder's IPC filters have priority. 76 // embedder's IPC filters have priority.
74 virtual void BrowserRenderProcessHostCreated( 77 virtual void BrowserRenderProcessHostCreated(
75 BrowserRenderProcessHost* host) = 0; 78 BrowserRenderProcessHost* host) = 0;
76 79
77 // Notifies that a PluginProcessHost has been created. This is called 80 // Notifies that a PluginProcessHost has been created. This is called
78 // before the content layer adds its own message filters, so that the 81 // before the content layer adds its own message filters, so that the
79 // embedder's IPC filters have priority. 82 // embedder's IPC filters have priority.
80 virtual void PluginProcessHostCreated(PluginProcessHost* host) = 0; 83 virtual void PluginProcessHostCreated(PluginProcessHost* host) = 0;
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
272 // This is called on a worker thread. 275 // This is called on a worker thread.
273 virtual 276 virtual
274 crypto::CryptoModuleBlockingPasswordDelegate* GetCryptoPasswordDelegate( 277 crypto::CryptoModuleBlockingPasswordDelegate* GetCryptoPasswordDelegate(
275 const GURL& url) = 0; 278 const GURL& url) = 0;
276 #endif 279 #endif
277 }; 280 };
278 281
279 } // namespace content 282 } // namespace content
280 283
281 #endif // CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_ 284 #endif // CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698