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

Unified Diff: ppapi/native_client/src/trusted/plugin/service_runtime.cc

Issue 9390028: Remove browser support for non-PPAPI nexes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 10 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: ppapi/native_client/src/trusted/plugin/service_runtime.cc
===================================================================
--- ppapi/native_client/src/trusted/plugin/service_runtime.cc (revision 121986)
+++ ppapi/native_client/src/trusted/plugin/service_runtime.cc (working copy)
@@ -33,12 +33,6 @@
#include "native_client/src/trusted/desc/nrd_xfer_effector.h"
#include "native_client/src/trusted/handle_pass/browser_handle.h"
#include "native_client/src/trusted/nonnacl_util/sel_ldr_launcher.h"
-
-// browser_interface includes portability.h for uintptr_t etc, but it
-// also transitively includes windows.h, where PostMessage gets
-// defined as a preprocessor symbol
-#include "native_client/src/trusted/plugin/browser_interface.h"
-
#include "native_client/src/trusted/plugin/manifest.h"
// This is here due to a Windows API collision; plugin.h through
@@ -49,7 +43,6 @@
#endif
#include "native_client/src/trusted/plugin/plugin.h"
#include "native_client/src/trusted/plugin/plugin_error.h"
-#include "native_client/src/trusted/plugin/scriptable_handle.h"
#include "native_client/src/trusted/plugin/srpc_client.h"
#include "native_client/src/trusted/plugin/utility.h"
@@ -143,8 +136,7 @@
NaClLog(4,
"PluginReverseInterface::Log_MainThreadContinuation(%s)\n",
p->message.c_str());
- plugin_->browser_interface()->AddToConsole(static_cast<Plugin*>(plugin_),
- p->message);
+ plugin_->AddToConsole(p->message);
}
void PluginReverseInterface::PostMessage_MainThreadContinuation(
PostMessageResource* p,
@@ -387,11 +379,8 @@
pp::CompletionCallback crash_cb)
: plugin_(plugin),
should_report_uma_(should_report_uma),
- browser_interface_(plugin->browser_interface()),
reverse_service_(NULL),
subprocess_(NULL),
- async_receive_desc_(NULL),
- async_send_desc_(NULL),
anchor_(new nacl::WeakRefAnchor()),
rev_interface_(new PluginReverseInterface(anchor_, plugin,
manifest,
@@ -528,10 +517,6 @@
return false;
}
- async_receive_desc_.reset(
- plugin()->wrapper_factory()->MakeImcSock(sockets[1]));
- async_send_desc_.reset(plugin()->wrapper_factory()->MakeImcSock(sockets[2]));
-
subprocess_.reset(tmp_subprocess.release());
if (!InitCommunication(nacl_desc, error_info)) {
subprocess_.reset(NULL);
@@ -552,7 +537,7 @@
} else {
PLUGIN_PRINTF(("ServiceRuntime::SetupAppChannel (conect_desc=%p)\n",
static_cast<void*>(connect_desc)));
- SrpcClient* srpc_client = SrpcClient::New(plugin(), connect_desc);
+ SrpcClient* srpc_client = SrpcClient::New(connect_desc);
PLUGIN_PRINTF(("ServiceRuntime::SetupAppChannel (srpc_client=%p)\n",
static_cast<void*>(srpc_client)));
delete connect_desc;

Powered by Google App Engine
This is Rietveld 408576698