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

Side by Side Diff: chrome/browser/nacl_host/nacl_process_host.h

Issue 16154005: Remove dependency from NaCl code on chrome_constants. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 6 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
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 CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 5 #ifndef CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
6 #define CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 6 #define CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
7 7
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 9
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 #elif defined(OS_LINUX) 105 #elif defined(OS_LINUX)
106 bool LaunchNaClGdb(base::ProcessId pid); 106 bool LaunchNaClGdb(base::ProcessId pid);
107 void OnNaClGdbAttached(); 107 void OnNaClGdbAttached();
108 #endif 108 #endif
109 #if defined(OS_POSIX) 109 #if defined(OS_POSIX)
110 // Create bound TCP socket in the browser process so that the NaCl GDB debug 110 // Create bound TCP socket in the browser process so that the NaCl GDB debug
111 // stub can use it to accept incoming connections even when the Chrome sandbox 111 // stub can use it to accept incoming connections even when the Chrome sandbox
112 // is enabled. 112 // is enabled.
113 SocketDescriptor GetDebugStubSocketHandle(); 113 SocketDescriptor GetDebugStubSocketHandle();
114 #endif 114 #endif
115 #if defined(OS_WIN)
116 // Get path to NaCl loader on local disk if possible.
Mark Seaborn 2013/05/31 19:42:30 Nit: "disk" -> "filesystem", since a lot of system
117 // |exe_path| does not change if the method fails.
118 bool GetNaclAppPath(base::FilePath& exe_path);
dmichael (off chromium) 2013/05/31 19:42:18 style nit: We use pointers for out-params in Chrom
Mark Seaborn 2013/05/31 19:42:30 This should be spelled as "NaCl", not "Nacl". Can
119 #endif
115 // Get path to manifest on local disk if possible. 120 // Get path to manifest on local disk if possible.
116 base::FilePath GetManifestPath(); 121 base::FilePath GetManifestPath();
117 bool LaunchSelLdr(); 122 bool LaunchSelLdr();
118 123
119 // BrowserChildProcessHostDelegate implementation: 124 // BrowserChildProcessHostDelegate implementation:
120 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE; 125 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
121 virtual void OnProcessLaunched() OVERRIDE; 126 virtual void OnProcessLaunched() OVERRIDE;
122 127
123 void OnResourcesReady(); 128 void OnResourcesReady();
124 129
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
220 PluginListener ipc_plugin_listener_; 225 PluginListener ipc_plugin_listener_;
221 // Browser host for plugin process. 226 // Browser host for plugin process.
222 scoped_ptr<content::BrowserPpapiHost> ppapi_host_; 227 scoped_ptr<content::BrowserPpapiHost> ppapi_host_;
223 228
224 int render_view_id_; 229 int render_view_id_;
225 230
226 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost); 231 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost);
227 }; 232 };
228 233
229 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 234 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698