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

Unified Diff: content/browser/ppapi_plugin_process_host.h

Issue 10014013: Add a hang monitor for Pepper plugins (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix leak Created 8 years, 8 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/ppapi_plugin_process_host.h
diff --git a/content/browser/ppapi_plugin_process_host.h b/content/browser/ppapi_plugin_process_host.h
index 6035b2ed05cae478616624349888df14a061ae02..64d76828130e3b6d81892d403fcdd2cdf51d4a82 100644
--- a/content/browser/ppapi_plugin_process_host.h
+++ b/content/browser/ppapi_plugin_process_host.h
@@ -42,9 +42,11 @@ class PpapiPluginProcessHost : public content::BrowserChildProcessHostDelegate,
// Called when the channel is asynchronously opened to the plugin or on
// error. On error, the parameters should be:
// base::kNullProcessHandle
- // IPC::ChannelHandle()
+ // IPC::ChannelHandle(),
+ // 0
virtual void OnChannelOpened(base::ProcessHandle plugin_process_handle,
- const IPC::ChannelHandle& channel_handle) = 0;
+ const IPC::ChannelHandle& channel_handle,
+ int plugin_child_id) = 0;
};
class PluginClient : public Client {
@@ -119,9 +121,6 @@ class PpapiPluginProcessHost : public content::BrowserChildProcessHostDelegate,
const bool is_broker_;
- // The unique id created for the process.
- int process_id_;
-
scoped_ptr<BrowserChildProcessHostImpl> process_;
DISALLOW_COPY_AND_ASSIGN(PpapiPluginProcessHost);

Powered by Google App Engine
This is Rietveld 408576698