Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2011 Google Inc. All rights reserved. | 2 * Copyright (C) 2011 Google Inc. All rights reserved. |
| 3 * | 3 * |
| 4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
| 5 * modification, are permitted provided that the following conditions are | 5 * modification, are permitted provided that the following conditions are |
| 6 * met: | 6 * met: |
| 7 * | 7 * |
| 8 * * Redistributions of source code must retain the above copyright | 8 * * Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * * Redistributions in binary form must reproduce the above | 10 * * Redistributions in binary form must reproduce the above |
| (...skipping 14 matching lines...) Expand all Loading... | |
| 25 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY | 25 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
| 26 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 26 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
| 27 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 27 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 29 */ | 29 */ |
| 30 | 30 |
| 31 #ifndef V8RecursionScope_h | 31 #ifndef V8RecursionScope_h |
| 32 #define V8RecursionScope_h | 32 #define V8RecursionScope_h |
| 33 | 33 |
| 34 #include "bindings/v8/V8PerIsolateData.h" | 34 #include "bindings/v8/V8PerIsolateData.h" |
| 35 #include "core/dom/ExecutionContext.h" | |
| 36 #include "wtf/Noncopyable.h" | 35 #include "wtf/Noncopyable.h" |
| 37 | 36 |
| 38 namespace WebCore { | 37 namespace WebCore { |
| 39 | 38 |
| 39 class ExecutionContext; | |
| 40 | |
| 40 // C++ calls into script contexts which are "owned" by WebKit (created in a | 41 // C++ calls into script contexts which are "owned" by WebKit (created in a |
| 41 // process where WebKit.cpp initializes v8) must declare their type: | 42 // process where WebKit.cpp initializes v8) must declare their type: |
| 42 // | 43 // |
| 43 // 1. Calls into page/author script from a frame | 44 // 1. Calls into page/author script from a frame |
| 44 // 2. Calls into page/author script from a worker | 45 // 2. Calls into page/author script from a worker |
| 45 // 3. Calls into internal script (typically setup/teardown work) | 46 // 3. Calls into internal script (typically setup/teardown work) |
| 46 // | 47 // |
| 47 // Debug-time checking of this is enforced via this class. | 48 // Debug-time checking of this is enforced via this class. |
| 48 // | 49 // |
| 49 // Calls of type (1) should generally go through ScriptController, as inspector | 50 // Calls of type (1) should generally go through ScriptController, as inspector |
| 50 // instrumentation is needed. ScriptController allocates V8RecursionScope for yo u. | 51 // instrumentation is needed. ScriptController allocates V8RecursionScope for yo u. |
| 51 // Calls of type (2) should always stack-allocate a V8RecursionScope in the same | 52 // Calls of type (2) should always stack-allocate a V8RecursionScope in the same |
| 52 // block as the call into script. Calls of type (3) should stack allocate a | 53 // block as the call into script. Calls of type (3) should stack allocate a |
| 53 // V8RecursionScope::MicrotaskSuppression -- this skips work that is spec'd to | 54 // V8RecursionScope::MicrotaskSuppression -- this skips work that is spec'd to |
| 54 // happen at the end of the outer-most script stack frame of calls into page scr ipt: | 55 // happen at the end of the outer-most script stack frame of calls into page scr ipt: |
| 55 // | 56 // |
| 56 // http://www.whatwg.org/specs/web-apps/current-work/#perform-a-microtask-checkp oint | 57 // http://www.whatwg.org/specs/web-apps/current-work/#perform-a-microtask-checkp oint |
| 57 class V8RecursionScope { | 58 class V8RecursionScope { |
| 58 WTF_MAKE_NONCOPYABLE(V8RecursionScope); | 59 WTF_MAKE_NONCOPYABLE(V8RecursionScope); |
| 59 public: | 60 public: |
| 60 explicit V8RecursionScope(ExecutionContext* context) | 61 explicit V8RecursionScope(ExecutionContext* context) |
| 61 : m_isDocumentContext(context && context->isDocument()) | 62 : m_context(context) |
| 62 { | 63 { |
| 63 V8PerIsolateData::current()->incrementRecursionLevel(); | 64 V8PerIsolateData::current()->incrementRecursionLevel(); |
| 64 } | 65 } |
| 65 | 66 |
| 66 ~V8RecursionScope() | 67 ~V8RecursionScope() |
| 67 { | 68 { |
| 68 if (!V8PerIsolateData::current()->decrementRecursionLevel()) | 69 if (!V8PerIsolateData::current()->decrementRecursionLevel()) |
| 69 didLeaveScriptContext(); | 70 didLeaveScriptContext(); |
| 70 } | 71 } |
| 71 | 72 |
| (...skipping 21 matching lines...) Expand all Loading... | |
| 93 ~MicrotaskSuppression() | 94 ~MicrotaskSuppression() |
| 94 { | 95 { |
| 95 #ifndef NDEBUG | 96 #ifndef NDEBUG |
| 96 V8PerIsolateData::current()->decrementInternalScriptRecursionLevel() ; | 97 V8PerIsolateData::current()->decrementInternalScriptRecursionLevel() ; |
| 97 #endif | 98 #endif |
| 98 } | 99 } |
| 99 }; | 100 }; |
| 100 | 101 |
| 101 private: | 102 private: |
| 102 void didLeaveScriptContext(); | 103 void didLeaveScriptContext(); |
| 104 bool isStopped(); | |
| 103 | 105 |
| 104 bool m_isDocumentContext; | 106 ExecutionContext* m_context; |
|
adamk
2014/03/28 17:55:05
Can you explain why it's safe to store the context
| |
| 105 }; | 107 }; |
| 106 | 108 |
| 107 } // namespace WebCore | 109 } // namespace WebCore |
| 108 | 110 |
| 109 #endif // V8RecursionScope_h | 111 #endif // V8RecursionScope_h |
| OLD | NEW |