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

Side by Side Diff: base/threading/thread_restrictions.h

Issue 1566573002: Fix race on mojo message pump shutdown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mojo-waitset-message-pump
Patch Set: Revert stuff. Created 4 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 | « base/message_loop/message_loop.cc ('k') | mojo/message_pump/handle_watcher.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) 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 BASE_THREADING_THREAD_RESTRICTIONS_H_ 5 #ifndef BASE_THREADING_THREAD_RESTRICTIONS_H_
6 #define BASE_THREADING_THREAD_RESTRICTIONS_H_ 6 #define BASE_THREADING_THREAD_RESTRICTIONS_H_
7 7
8 #include "base/base_export.h" 8 #include "base/base_export.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 10
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 namespace dbus { 49 namespace dbus {
50 class Bus; 50 class Bus;
51 } 51 }
52 namespace disk_cache { 52 namespace disk_cache {
53 class BackendImpl; 53 class BackendImpl;
54 class InFlightIO; 54 class InFlightIO;
55 } 55 }
56 namespace gles2 { 56 namespace gles2 {
57 class CommandBufferClientImpl; 57 class CommandBufferClientImpl;
58 } 58 }
59 namespace mojo {
60 namespace common {
61 class MessagePumpMojo;
62 }
63 }
59 namespace net { 64 namespace net {
60 class NetworkChangeNotifierMac; 65 class NetworkChangeNotifierMac;
61 namespace internal { 66 namespace internal {
62 class AddressTrackerLinux; 67 class AddressTrackerLinux;
63 } 68 }
64 } 69 }
65 70
66 namespace remoting { 71 namespace remoting {
67 class AutoThread; 72 class AutoThread;
68 } 73 }
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 friend class remoting::AutoThread; 197 friend class remoting::AutoThread;
193 friend class ui::WindowResizeHelperMac; 198 friend class ui::WindowResizeHelperMac;
194 friend class MessagePumpDefault; 199 friend class MessagePumpDefault;
195 friend class SequencedWorkerPool; 200 friend class SequencedWorkerPool;
196 friend class SimpleThread; 201 friend class SimpleThread;
197 friend class Thread; 202 friend class Thread;
198 friend class ThreadTestHelper; 203 friend class ThreadTestHelper;
199 friend class PlatformThread; 204 friend class PlatformThread;
200 friend class android::JavaHandlerThread; 205 friend class android::JavaHandlerThread;
201 friend class gles2::CommandBufferClientImpl; 206 friend class gles2::CommandBufferClientImpl;
207 friend class mojo::common::MessagePumpMojo;
202 208
203 // END ALLOWED USAGE. 209 // END ALLOWED USAGE.
204 // BEGIN USAGE THAT NEEDS TO BE FIXED. 210 // BEGIN USAGE THAT NEEDS TO BE FIXED.
205 friend class ::chromeos::BlockingMethodCaller; // http://crbug.com/125360 211 friend class ::chromeos::BlockingMethodCaller; // http://crbug.com/125360
206 friend class ::chromeos::system::StatisticsProviderImpl; // http://crbug.com/ 125385 212 friend class ::chromeos::system::StatisticsProviderImpl; // http://crbug.com/ 125385
207 friend class chrome_browser_net::Predictor; // http://crbug.com/78451 213 friend class chrome_browser_net::Predictor; // http://crbug.com/78451
208 friend class 214 friend class
209 content::BrowserGpuChannelHostFactory; // http://crbug.com/125248 215 content::BrowserGpuChannelHostFactory; // http://crbug.com/125248
210 friend class 216 friend class
211 content::BrowserGpuMemoryBufferManager; // http://crbug.com/420368 217 content::BrowserGpuMemoryBufferManager; // http://crbug.com/420368
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
245 251
246 DISALLOW_COPY_AND_ASSIGN(ScopedAllowWait); 252 DISALLOW_COPY_AND_ASSIGN(ScopedAllowWait);
247 }; 253 };
248 254
249 DISALLOW_IMPLICIT_CONSTRUCTORS(ThreadRestrictions); 255 DISALLOW_IMPLICIT_CONSTRUCTORS(ThreadRestrictions);
250 }; 256 };
251 257
252 } // namespace base 258 } // namespace base
253 259
254 #endif // BASE_THREADING_THREAD_RESTRICTIONS_H_ 260 #endif // BASE_THREADING_THREAD_RESTRICTIONS_H_
OLDNEW
« no previous file with comments | « base/message_loop/message_loop.cc ('k') | mojo/message_pump/handle_watcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698