| OLD | NEW |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 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 MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ | 5 #ifndef MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ |
| 6 #define MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ | 6 #define MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ |
| 7 | 7 |
| 8 #include "base/macros.h" | 8 #include "base/macros.h" |
| 9 #include "base/threading/thread_restrictions.h" | 9 #include "base/threading/thread_restrictions.h" |
| 10 #include "mojo/public/cpp/bindings/bindings_export.h" | 10 #include "mojo/public/cpp/bindings/bindings_export.h" |
| 11 | 11 |
| 12 #if (!defined(NDEBUG) || defined(DCHECK_ALWAYS_ON)) | 12 #if (!defined(NDEBUG) || defined(DCHECK_ALWAYS_ON)) |
| 13 #define ENABLE_SYNC_CALL_RESTRICTIONS 1 | 13 #define ENABLE_SYNC_CALL_RESTRICTIONS 1 |
| 14 #else | 14 #else |
| 15 #define ENABLE_SYNC_CALL_RESTRICTIONS 0 | 15 #define ENABLE_SYNC_CALL_RESTRICTIONS 0 |
| 16 #endif | 16 #endif |
| 17 | 17 |
| 18 namespace ui { | 18 namespace ui { |
| 19 class GpuService; | 19 class GpuService; |
| 20 } | 20 } |
| 21 | 21 |
| 22 namespace aura { |
| 23 class GpuService; |
| 24 } |
| 25 |
| 22 namespace views { | 26 namespace views { |
| 23 class ClipboardMus; | 27 class ClipboardMus; |
| 24 } | 28 } |
| 25 | 29 |
| 26 namespace mojo { | 30 namespace mojo { |
| 27 | 31 |
| 28 // In some processes, sync calls are disallowed. For example, in the browser | 32 // In some processes, sync calls are disallowed. For example, in the browser |
| 29 // process we don't want any sync calls to child processes for performance, | 33 // process we don't want any sync calls to child processes for performance, |
| 30 // security and stability reasons. SyncCallRestrictions helps to enforce such | 34 // security and stability reasons. SyncCallRestrictions helps to enforce such |
| 31 // rules. | 35 // rules. |
| (...skipping 13 matching lines...) Expand all Loading... |
| 45 static void AssertSyncCallAllowed(); | 49 static void AssertSyncCallAllowed(); |
| 46 #else | 50 #else |
| 47 // Inline the empty definitions of functions so that they can be compiled out. | 51 // Inline the empty definitions of functions so that they can be compiled out. |
| 48 static void AssertSyncCallAllowed() {} | 52 static void AssertSyncCallAllowed() {} |
| 49 #endif | 53 #endif |
| 50 | 54 |
| 51 private: | 55 private: |
| 52 // DO NOT ADD ANY OTHER FRIEND STATEMENTS, talk to mojo/OWNERS first. | 56 // DO NOT ADD ANY OTHER FRIEND STATEMENTS, talk to mojo/OWNERS first. |
| 53 // BEGIN ALLOWED USAGE. | 57 // BEGIN ALLOWED USAGE. |
| 54 friend class ui::GpuService; // http://crbug.com/620058 | 58 friend class ui::GpuService; // http://crbug.com/620058 |
| 59 friend class aura::GpuService; // http://crbug.com/620058 |
| 55 // END ALLOWED USAGE. | 60 // END ALLOWED USAGE. |
| 56 | 61 |
| 57 // BEGIN USAGE THAT NEEDS TO BE FIXED. | 62 // BEGIN USAGE THAT NEEDS TO BE FIXED. |
| 58 // In the non-mus case, we called blocking OS functions in the ui::Clipboard | 63 // In the non-mus case, we called blocking OS functions in the ui::Clipboard |
| 59 // implementation which weren't caught by sync call restrictions. Our blocking | 64 // implementation which weren't caught by sync call restrictions. Our blocking |
| 60 // calls to mus, however, are. | 65 // calls to mus, however, are. |
| 61 friend class views::ClipboardMus; | 66 friend class views::ClipboardMus; |
| 62 // END USAGE THAT NEEDS TO BE FIXED. | 67 // END USAGE THAT NEEDS TO BE FIXED. |
| 63 | 68 |
| 64 #if ENABLE_SYNC_CALL_RESTRICTIONS | 69 #if ENABLE_SYNC_CALL_RESTRICTIONS |
| (...skipping 21 matching lines...) Expand all Loading... |
| 86 | 91 |
| 87 DISALLOW_COPY_AND_ASSIGN(ScopedAllowSyncCall); | 92 DISALLOW_COPY_AND_ASSIGN(ScopedAllowSyncCall); |
| 88 }; | 93 }; |
| 89 | 94 |
| 90 DISALLOW_IMPLICIT_CONSTRUCTORS(SyncCallRestrictions); | 95 DISALLOW_IMPLICIT_CONSTRUCTORS(SyncCallRestrictions); |
| 91 }; | 96 }; |
| 92 | 97 |
| 93 } // namespace mojo | 98 } // namespace mojo |
| 94 | 99 |
| 95 #endif // MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ | 100 #endif // MOJO_PUBLIC_CPP_BINDINGS_SYNC_CALL_RESTRICTIONS_H_ |
| OLD | NEW |