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

Side by Side Diff: content/browser/browser_process_sub_thread.h

Issue 9221008: Move cleanup that happens on IO thread destruction to content, so that not every embedder has to ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | content/browser/browser_process_sub_thread.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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_ 5 #ifndef CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_
6 #define CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_ 6 #define CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_
7 #pragma once 7 #pragma once
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "content/browser/browser_thread_impl.h" 10 #include "content/browser/browser_thread_impl.h"
(...skipping 17 matching lines...) Expand all
28 class CONTENT_EXPORT BrowserProcessSubThread : public BrowserThreadImpl { 28 class CONTENT_EXPORT BrowserProcessSubThread : public BrowserThreadImpl {
29 public: 29 public:
30 explicit BrowserProcessSubThread(BrowserThread::ID identifier); 30 explicit BrowserProcessSubThread(BrowserThread::ID identifier);
31 virtual ~BrowserProcessSubThread(); 31 virtual ~BrowserProcessSubThread();
32 32
33 protected: 33 protected:
34 virtual void Init() OVERRIDE; 34 virtual void Init() OVERRIDE;
35 virtual void CleanUp() OVERRIDE; 35 virtual void CleanUp() OVERRIDE;
36 36
37 private: 37 private:
38 // These methods encapsulate cleanup that needs to happen on the IO thread
39 // before/after we call the embedder's CleanUp function.
40 void IOThreadPreCleanUp();
41 void IOThreadPostCleanUp();
42
38 // Each specialized thread has its own notification service. 43 // Each specialized thread has its own notification service.
39 // Note: We don't use scoped_ptr because the destructor runs on the wrong 44 // Note: We don't use scoped_ptr because the destructor runs on the wrong
40 // thread. 45 // thread.
41 NotificationService* notification_service_; 46 NotificationService* notification_service_;
42 47
43 DISALLOW_COPY_AND_ASSIGN(BrowserProcessSubThread); 48 DISALLOW_COPY_AND_ASSIGN(BrowserProcessSubThread);
44 }; 49 };
45 50
46 } // namespace content 51 } // namespace content
47 52
48 #endif // CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_ 53 #endif // CONTENT_BROWSER_BROWSER_PROCESS_SUB_THREAD_H_
OLDNEW
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | content/browser/browser_process_sub_thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698