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

Side by Side Diff: Source/bindings/v8/V8RecursionScope.h

Issue 209853010: [ABANDONED] Enable V8 Promises (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: done? Created 6 years, 9 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
OLDNEW
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
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
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
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698