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

Side by Side Diff: chrome/browser/browser_process_impl.h

Issue 16554: WaitableEvent (Closed)
Patch Set: Addresssing darin's comments (round 2) Created 11 years, 11 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
« no previous file with comments | « chrome/browser/browser_process.h ('k') | chrome/browser/browser_process_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 // When each service is created, we set a flag indicating this. At this point, 5 // When each service is created, we set a flag indicating this. At this point,
6 // the service initialization could fail or succeed. This allows us to remember 6 // the service initialization could fail or succeed. This allows us to remember
7 // if we tried to create a service, and not try creating it over and over if 7 // if we tried to create a service, and not try creating it over and over if
8 // the creation failed. 8 // the creation failed.
9 9
10 #ifndef CHROME_BROWSER_BROWSER_PROCESS_IMPL_H__ 10 #ifndef CHROME_BROWSER_BROWSER_PROCESS_IMPL_H__
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 return google_url_tracker_.get(); 160 return google_url_tracker_.get();
161 } 161 }
162 162
163 virtual const std::wstring& GetApplicationLocale(); 163 virtual const std::wstring& GetApplicationLocale();
164 164
165 virtual MemoryModel memory_model() { 165 virtual MemoryModel memory_model() {
166 DCHECK(CalledOnValidThread()); 166 DCHECK(CalledOnValidThread());
167 return memory_model_; 167 return memory_model_;
168 } 168 }
169 169
170 virtual HANDLE shutdown_event() { return shutdown_event_; } 170 virtual base::WaitableEvent* shutdown_event() { return shutdown_event_; }
171 171
172 private: 172 private:
173 void CreateResourceDispatcherHost(); 173 void CreateResourceDispatcherHost();
174 void CreatePrefService(); 174 void CreatePrefService();
175 void CreateMetricsService(); 175 void CreateMetricsService();
176 void CreateIOThread(); 176 void CreateIOThread();
177 void CreateFileThread(); 177 void CreateFileThread();
178 void CreateDBThread(); 178 void CreateDBThread();
179 void CreateSafeBrowsingThread(); 179 void CreateSafeBrowsingThread();
180 void CreateTemplateURLModel(); 180 void CreateTemplateURLModel();
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 scoped_ptr<printing::PrintJobManager> print_job_manager_; 235 scoped_ptr<printing::PrintJobManager> print_job_manager_;
236 236
237 std::wstring locale_; 237 std::wstring locale_;
238 238
239 MemoryModel memory_model_; 239 MemoryModel memory_model_;
240 240
241 bool checked_for_new_frames_; 241 bool checked_for_new_frames_;
242 bool using_new_frames_; 242 bool using_new_frames_;
243 243
244 // An event that notifies when we are shutting-down. 244 // An event that notifies when we are shutting-down.
245 HANDLE shutdown_event_; 245 base::WaitableEvent* shutdown_event_;
246 246
247 DISALLOW_EVIL_CONSTRUCTORS(BrowserProcessImpl); 247 DISALLOW_EVIL_CONSTRUCTORS(BrowserProcessImpl);
248 }; 248 };
249 249
250 #endif // CHROME_BROWSER_BROWSER_PROCESS_IMPL_H__ 250 #endif // CHROME_BROWSER_BROWSER_PROCESS_IMPL_H__
251 251
OLDNEW
« no previous file with comments | « chrome/browser/browser_process.h ('k') | chrome/browser/browser_process_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698