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

Unified Diff: content/browser/ppapi_plugin_process_host.cc

Issue 11450025: Revert 171389 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years 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.cc
diff --git a/content/browser/ppapi_plugin_process_host.cc b/content/browser/ppapi_plugin_process_host.cc
index 655c5133c76d9cb04629289d24c05735c16f354a..4d0213bd0400a60fa7d67b4dcd9c03cb17253979 100644
--- a/content/browser/ppapi_plugin_process_host.cc
+++ b/content/browser/ppapi_plugin_process_host.cc
@@ -172,13 +172,10 @@ PpapiPluginProcessHost::PpapiPluginProcessHost(
permissions_,
host_resolver);
- host_impl_.reset(new BrowserPpapiHostImpl(this, permissions_));
-
- file_filter_ = new PepperTrustedFileMessageFilter(
- process_->GetData().id, info.name, profile_data_directory);
+ host_impl_.reset(new BrowserPpapiHostImpl(this, permissions_, info.name,
+ profile_data_directory, process_->GetData().id));
process_->GetHost()->AddFilter(filter_.get());
- process_->GetHost()->AddFilter(file_filter_.get());
process_->GetHost()->AddFilter(host_impl_->message_filter());
GetContentClient()->browser()->DidCreatePpapiPlugin(host_impl_.get());
@@ -190,7 +187,12 @@ PpapiPluginProcessHost::PpapiPluginProcessHost()
PROCESS_TYPE_PPAPI_BROKER, this));
ppapi::PpapiPermissions permissions; // No permissions.
- host_impl_.reset(new BrowserPpapiHostImpl(this, permissions));
+ // The plugin name and profile data directory shouldn't be needed for the
+ // broker.
+ std::string plugin_name;
+ FilePath profile_data_directory;
+ host_impl_.reset(new BrowserPpapiHostImpl(this, permissions, plugin_name,
+ profile_data_directory, process_->GetData().id));
}
bool PpapiPluginProcessHost::Init(const PepperPluginInfo& info) {
« no previous file with comments | « content/browser/ppapi_plugin_process_host.h ('k') | content/browser/renderer_host/pepper/browser_ppapi_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698