From 0c94bdf1740d0836fe7ad1fe9123b704682c9d70 Mon Sep 17 00:00:00 2001 From: "bradnelson@google.com" Date: Sat, 17 May 2014 01:03:21 +0000 Subject: Remove knowledge of nacl from content. Content should not have knowledge of NaCl. Moving per process NaCl debug stub port information to NaClBrowser. More explicitly distiquish between the case where the debug port is not yet known versus when it is unused. Switch debug port to being a per-process value (as it is). This relands r270907 with a fix to make it work with disable_nacl=1 and therefore ASAN. BUG=None TEST=trybots R=jam@chromium.org,sehr@chromium.org Review URL: https://codereview.chromium.org/286143010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271133 0039d316-1c4b-4281-b951-d872f2087c98 --- content/browser/browser_child_process_host_impl.cc | 5 ----- content/browser/browser_child_process_host_impl.h | 1 - content/public/browser/browser_child_process_host.h | 3 --- content/public/browser/child_process_data.h | 5 +---- 4 files changed, 1 insertion(+), 13 deletions(-) (limited to 'content') diff --git a/content/browser/browser_child_process_host_impl.cc b/content/browser/browser_child_process_host_impl.cc index 97058a6..dd7d600 100644 --- a/content/browser/browser_child_process_host_impl.cc +++ b/content/browser/browser_child_process_host_impl.cc @@ -177,11 +177,6 @@ base::ProcessHandle BrowserChildProcessHostImpl::GetHandle() const { return child_process_->GetHandle(); } -void BrowserChildProcessHostImpl::SetNaClDebugStubPort(int port) { - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); - data_.nacl_debug_stub_port = port; -} - void BrowserChildProcessHostImpl::SetName(const base::string16& name) { DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); data_.name = name; diff --git a/content/browser/browser_child_process_host_impl.h b/content/browser/browser_child_process_host_impl.h index 0f7a3df..b196551 100644 --- a/content/browser/browser_child_process_host_impl.h +++ b/content/browser/browser_child_process_host_impl.h @@ -53,7 +53,6 @@ class CONTENT_EXPORT BrowserChildProcessHostImpl virtual ChildProcessHost* GetHost() const OVERRIDE; virtual base::TerminationStatus GetTerminationStatus( bool known_dead, int* exit_code) OVERRIDE; - virtual void SetNaClDebugStubPort(int port) OVERRIDE; virtual void SetName(const base::string16& name) OVERRIDE; virtual void SetHandle(base::ProcessHandle handle) OVERRIDE; diff --git a/content/public/browser/browser_child_process_host.h b/content/public/browser/browser_child_process_host.h index a44b8d3..078eb86 100644 --- a/content/public/browser/browser_child_process_host.h +++ b/content/public/browser/browser_child_process_host.h @@ -71,9 +71,6 @@ class CONTENT_EXPORT BrowserChildProcessHost : public IPC::Sender { // this object. virtual void SetHandle(base::ProcessHandle handle) = 0; - // Set the nacl debug stub port of the process. - virtual void SetNaClDebugStubPort(int port) = 0; - #if defined(OS_MACOSX) && !defined(OS_IOS) // Returns a PortProvider used to get process metrics for child processes. static base::ProcessMetrics::PortProvider* GetPortProvider(); diff --git a/content/public/browser/child_process_data.h b/content/public/browser/child_process_data.h index 2807657..3683be9 100644 --- a/content/public/browser/child_process_data.h +++ b/content/public/browser/child_process_data.h @@ -28,11 +28,8 @@ struct ChildProcessData { // The handle to the process. base::ProcessHandle handle; - int nacl_debug_stub_port; - explicit ChildProcessData(int process_type) - : process_type(process_type), id(0), handle(base::kNullProcessHandle), - nacl_debug_stub_port(0) { + : process_type(process_type), id(0), handle(base::kNullProcessHandle) { } }; -- cgit v1.1