diff options
author | gregoryd@google.com <gregoryd@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-26 10:53:26 +0000 |
---|---|---|
committer | gregoryd@google.com <gregoryd@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-26 10:53:26 +0000 |
commit | 8939b46c28247a9c7e2f349020f61ea7038e16ee (patch) | |
tree | 8ed6ccd597fddd797541111d8e585eeed3a107ba /chrome/common | |
parent | eb5eaec3e3e50d0effcfb53f629e0c93c0fa119c (diff) | |
download | chromium_src-8939b46c28247a9c7e2f349020f61ea7038e16ee.zip chromium_src-8939b46c28247a9c7e2f349020f61ea7038e16ee.tar.gz chromium_src-8939b46c28247a9c7e2f349020f61ea7038e16ee.tar.bz2 |
Revert 27324 - First step towards NaClChrome integration:1. NaCl plugin becomes a builtin plugin in Chrome and runs in the renderer process.2. Most of the changes are related to launching the NaCl process (that loads and runs the NaCl module) and establishing the initial communication between that process and the NaCl plugin.3. Command line flag "internalnacl" is required to enable the builtin NaCl plugin. NaCl still cannot run in Chrome sandbox, the flag automatically disables the sandboxCommitted: http://src.chromium.org/viewvc/chrome?view=rev&revision=27315
Review URL: http://codereview.chromium.org/153002
TBR=bradnelson@google.com
Review URL: http://codereview.chromium.org/244017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27326 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
-rw-r--r-- | chrome/common/child_process_info.h | 1 | ||||
-rw-r--r-- | chrome/common/chrome_switches.cc | 6 | ||||
-rw-r--r-- | chrome/common/chrome_switches.h | 3 | ||||
-rw-r--r-- | chrome/common/nacl_messages.h | 19 | ||||
-rw-r--r-- | chrome/common/nacl_messages_internal.h | 18 | ||||
-rw-r--r-- | chrome/common/nacl_types.h | 21 | ||||
-rw-r--r-- | chrome/common/render_messages_internal.h | 9 | ||||
-rw-r--r-- | chrome/common/sandbox_init_wrapper.cc | 1 |
8 files changed, 0 insertions, 78 deletions
diff --git a/chrome/common/child_process_info.h b/chrome/common/child_process_info.h index c99b2c0..250ff57 100644 --- a/chrome/common/child_process_info.h +++ b/chrome/common/child_process_info.h @@ -17,7 +17,6 @@ class ChildProcessInfo { RENDER_PROCESS, PLUGIN_PROCESS, WORKER_PROCESS, - NACL_PROCESS, UTILITY_PROCESS, PROFILE_IMPORT_PROCESS, ZYGOTE_PROCESS, diff --git a/chrome/common/chrome_switches.cc b/chrome/common/chrome_switches.cc index 7fa5db0..14aea58 100644 --- a/chrome/common/chrome_switches.cc +++ b/chrome/common/chrome_switches.cc @@ -71,9 +71,6 @@ const wchar_t kPluginProcess[] = L"plugin"; // Causes the process to run as a worker subprocess. const wchar_t kWorkerProcess[] = L"worker"; -// Causes the process to run as a NativeClient's sel_ldr subprocess. -const wchar_t kNaClProcess[] = L"nacl"; - // Causes the process to run as a utility subprocess. const wchar_t kUtilityProcess[] = L"utility"; @@ -370,9 +367,6 @@ const wchar_t kEnableRemoteFonts[] = L"enable-remote-fonts"; // Use the low fragmentation heap for the CRT. const wchar_t kUseLowFragHeapCrt[] = L"use-lf-heap"; -// Runs the Native Client inside the renderer process. -const wchar_t kInternalNaCl[] = L"internal-nacl"; - #ifndef NDEBUG // Debug only switch to specify which gears plugin dll to load. const wchar_t kGearsPluginPathOverride[] = L"gears-plugin-path"; diff --git a/chrome/common/chrome_switches.h b/chrome/common/chrome_switches.h index d5f7b9a..9ed753c 100644 --- a/chrome/common/chrome_switches.h +++ b/chrome/common/chrome_switches.h @@ -30,7 +30,6 @@ extern const wchar_t kZygoteProcess[]; extern const wchar_t kBrowserSubprocessPath[]; extern const wchar_t kPluginProcess[]; extern const wchar_t kWorkerProcess[]; -extern const wchar_t kNaClProcess[]; extern const wchar_t kUtilityProcess[]; extern const wchar_t kProfileImportProcess[]; extern const wchar_t kSingleProcess[]; @@ -134,8 +133,6 @@ extern const wchar_t kEnableRemoteFonts[]; extern const wchar_t kUseLowFragHeapCrt[]; -extern const wchar_t kInternalNaCl[]; - #ifndef NDEBUG extern const wchar_t kGearsPluginPathOverride[]; #endif diff --git a/chrome/common/nacl_messages.h b/chrome/common/nacl_messages.h deleted file mode 100644 index 8d360dd..0000000 --- a/chrome/common/nacl_messages.h +++ /dev/null @@ -1,19 +0,0 @@ -// Copyright (c) 2009 The Chromium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Defines messages between the browser and NaCl process. - -#ifndef CHROME_COMMON_NACL_MESSAGES_H_ -#define CHROME_COMMON_NACL_MESSAGES_H_ - -#include <string> - -#include "base/basictypes.h" -#include "ipc/ipc_message_utils.h" - -#define MESSAGES_INTERNAL_FILE "chrome/common/nacl_messages_internal.h" -#include "ipc/ipc_message_macros.h" - -#endif // CHROME_COMMON_NACL_MESSAGES_H_ - diff --git a/chrome/common/nacl_messages_internal.h b/chrome/common/nacl_messages_internal.h deleted file mode 100644 index 3794255..0000000 --- a/chrome/common/nacl_messages_internal.h +++ /dev/null @@ -1,18 +0,0 @@ -// Copyright (c) 2009 The Chromium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -#include "chrome/common/nacl_types.h" -#include "ipc/ipc_message_macros.h" - -//----------------------------------------------------------------------------- -// NaClProcess messages -// These are messages sent from the browser to the NaCl process. -IPC_BEGIN_MESSAGES(NaClProcess) - // Tells the NaCl process to start. - IPC_MESSAGE_CONTROL2(NaClProcessMsg_Start, - int /* descriptor id */, - nacl::FileDescriptor /* handle value */) - -IPC_END_MESSAGES(NaClProcess) - diff --git a/chrome/common/nacl_types.h b/chrome/common/nacl_types.h deleted file mode 100644 index 394edd3..0000000 --- a/chrome/common/nacl_types.h +++ /dev/null @@ -1,21 +0,0 @@ -// Copyright (c) 2009 The Chromium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Handle passing definitions for NaCl -#ifndef CHROME_COMMON_NACL_TYPES_H_ -#define CHROME_COMMON_NACL_TYPES_H_ - -// TODO(gregoryd): add a Windows definition for base::FileDescriptor, -// replace the macros with inline functions. -namespace nacl { -#if defined(OS_WIN) -typedef HANDLE FileDescriptor; -#define NATIVE_HANDLE(desc) (desc) -#elif defined(OS_POSIX) -typedef base::FileDescriptor FileDescriptor; -#define NATIVE_HANDLE(desc) ((desc).fd) -#endif -} - -#endif // CHROME_COMMON_NACL_TYPES_H_ diff --git a/chrome/common/render_messages_internal.h b/chrome/common/render_messages_internal.h index 11d14d5..4eec034 100644 --- a/chrome/common/render_messages_internal.h +++ b/chrome/common/render_messages_internal.h @@ -21,7 +21,6 @@ #include "base/values.h" #include "chrome/common/css_colors.h" #include "chrome/common/extensions/update_manifest.h" -#include "chrome/common/nacl_types.h" #include "chrome/common/notification_type.h" #include "chrome/common/transport_dib.h" #include "chrome/common/view_types.h" @@ -1090,14 +1089,6 @@ IPC_BEGIN_MESSAGES(ViewHost) IPC::ChannelHandle /* handle to channel */, WebPluginInfo /* info */) - // A renderer sends this to the browser process when it wants to start - // a new instance of the Native Client process. The browser will launch - // the process and return a handle to an IMC channel. - IPC_SYNC_MESSAGE_CONTROL1_1(ViewHostMsg_LaunchNaCl, - int /* channel number */, - nacl::FileDescriptor /* handle - one side - of a socket pair */) - #if defined(OS_LINUX) // A renderer sends this when it needs a browser-side widget for // hosting a windowed plugin. id is the XID of the plugin window, for which diff --git a/chrome/common/sandbox_init_wrapper.cc b/chrome/common/sandbox_init_wrapper.cc index 14006ed..c296ff8 100644 --- a/chrome/common/sandbox_init_wrapper.cc +++ b/chrome/common/sandbox_init_wrapper.cc @@ -27,7 +27,6 @@ void SandboxInitWrapper::InitializeSandbox(const CommandLine& command_line, if (!command_line.HasSwitch(switches::kNoSandbox)) { if ((process_type == switches::kRendererProcess) || (process_type == switches::kWorkerProcess) || - (process_type == switches::kNaClProcess) || (process_type == switches::kUtilityProcess) || (process_type == switches::kPluginProcess && command_line.HasSwitch(switches::kSafePlugins))) { |