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

Side by Side Diff: content/renderer/browser_plugin/old/browser_plugin.h

Issue 10830072: Browser Plugin: New Implementation (Renderer Side) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added lots more comments to messages and public members of BrowserPlugin Created 8 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_ 5 #ifndef CONTENT_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_
6 #define CONTENT_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_ 6 #define CONTENT_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_
7 7
8 #include "base/process.h" 8 #include "base/process.h"
9 #include "content/renderer/render_view_impl.h" 9 #include "content/renderer/render_view_impl.h"
10 #include "ipc/ipc_channel_handle.h" 10 #include "ipc/ipc_channel_handle.h"
(...skipping 20 matching lines...) Expand all
31 // To avoid blocking the embedder RenderView and to avoid introducing the 31 // To avoid blocking the embedder RenderView and to avoid introducing the
32 // potential for deadlock, BrowserPlugin attaches a placeholder that takes 32 // potential for deadlock, BrowserPlugin attaches a placeholder that takes
33 // place of the guest RenderView until the guest has established a connection 33 // place of the guest RenderView until the guest has established a connection
34 // with its embedder RenderView. This permits asynchronously loading of the 34 // with its embedder RenderView. This permits asynchronously loading of the
35 // guest while the embedder RenderView is permitted to continue to receive and 35 // guest while the embedder RenderView is permitted to continue to receive and
36 // process events. 36 // process events.
37 // 37 //
38 // Furthermore, certain navigations can swap to a new guest RenderView on an 38 // Furthermore, certain navigations can swap to a new guest RenderView on an
39 // different process. BrowserPlugin is the consistent facade that the embedder's 39 // different process. BrowserPlugin is the consistent facade that the embedder's
40 // WebKit instance talks to regardless of which process it's communicating with. 40 // WebKit instance talks to regardless of which process it's communicating with.
41 class BrowserPlugin { 41 class CONTENT_EXPORT BrowserPlugin {
42 public: 42 public:
43 // Creates a new WebViewPlugin with a BrowserPlugin as a delegate. 43 // Creates a new WebViewPlugin with a BrowserPlugin as a delegate.
44 static WebKit::WebPlugin* Create( 44 static WebKit::WebPlugin* Create(
45 RenderViewImpl* render_view, 45 RenderViewImpl* render_view,
46 WebKit::WebFrame* frame, 46 WebKit::WebFrame* frame,
47 const WebKit::WebPluginParams& params); 47 const WebKit::WebPluginParams& params);
48 48
49 static BrowserPlugin* FromID(int id); 49 static BrowserPlugin* FromID(int id);
50 50
51 webkit::WebViewPlugin* placeholder() { return placeholder_; } 51 webkit::WebViewPlugin* placeholder() { return placeholder_; }
(...skipping 28 matching lines...) Expand all
80 RenderViewImpl* render_view_; 80 RenderViewImpl* render_view_;
81 WebKit::WebPluginParams plugin_params_; 81 WebKit::WebPluginParams plugin_params_;
82 webkit::WebViewPlugin* placeholder_; 82 webkit::WebViewPlugin* placeholder_;
83 webkit::ppapi::WebPluginImpl* plugin_; 83 webkit::ppapi::WebPluginImpl* plugin_;
84 int id_; 84 int id_;
85 85
86 DISALLOW_COPY_AND_ASSIGN(BrowserPlugin); 86 DISALLOW_COPY_AND_ASSIGN(BrowserPlugin);
87 }; 87 };
88 88
89 #endif // CONTENT_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_ 89 #endif // CONTENT_RENDERER_BROWSER_PLUGIN_OLD_BROWSER_PLUGIN_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698