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

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

Issue 10383262: RefCounted types should not have public destructors, delegate cleanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 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 | Annotate | Revision Log
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 #pragma once 7 #pragma once
8 8
9 #include "build/build_config.h" 9 #include "build/build_config.h"
10 10
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 115
116 GURL manifest_url_; 116 GURL manifest_url_;
117 117
118 #if defined(OS_WIN) 118 #if defined(OS_WIN)
119 // This field becomes true when the broker successfully launched 119 // This field becomes true when the broker successfully launched
120 // the NaCl loader. 120 // the NaCl loader.
121 bool process_launched_by_broker_; 121 bool process_launched_by_broker_;
122 #elif defined(OS_LINUX) 122 #elif defined(OS_LINUX)
123 bool wait_for_nacl_gdb_; 123 bool wait_for_nacl_gdb_;
124 MessageLoopForIO::FileDescriptorWatcher nacl_gdb_watcher_; 124 MessageLoopForIO::FileDescriptorWatcher nacl_gdb_watcher_;
125 scoped_ptr<MessageLoopForIO::Watcher> nacl_gdb_watcher_delegate_; 125
126 class NaClGdbWatchDelegate;
127 scoped_ptr<NaClGdbWatchDelegate> nacl_gdb_watcher_delegate_;
126 #endif 128 #endif
127 // The ChromeRenderMessageFilter that requested this NaCl process. We use 129 // The ChromeRenderMessageFilter that requested this NaCl process. We use
128 // this for sending the reply once the process has started. 130 // this for sending the reply once the process has started.
129 scoped_refptr<ChromeRenderMessageFilter> chrome_render_message_filter_; 131 scoped_refptr<ChromeRenderMessageFilter> chrome_render_message_filter_;
130 132
131 // The reply message to send. We must always send this message when the 133 // The reply message to send. We must always send this message when the
132 // sub-process either succeeds or fails to unblock the renderer waiting for 134 // sub-process either succeeds or fails to unblock the renderer waiting for
133 // the reply. NULL when there is no reply to send. 135 // the reply. NULL when there is no reply to send.
134 IPC::Message* reply_msg_; 136 IPC::Message* reply_msg_;
135 #if defined(OS_WIN) 137 #if defined(OS_WIN)
(...skipping 11 matching lines...) Expand all
147 base::WeakPtrFactory<NaClProcessHost> weak_factory_; 149 base::WeakPtrFactory<NaClProcessHost> weak_factory_;
148 150
149 scoped_ptr<content::BrowserChildProcessHost> process_; 151 scoped_ptr<content::BrowserChildProcessHost> process_;
150 152
151 bool enable_exception_handling_; 153 bool enable_exception_handling_;
152 154
153 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost); 155 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost);
154 }; 156 };
155 157
156 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 158 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698