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

Side by Side Diff: Source/core/inspector/PromiseTracker.cpp

Issue 529723002: [WIP] Protocol for sending information about Promises to frontend. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Address comments + REBASE Created 6 years, 3 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 | « Source/core/inspector/PromiseTracker.h ('k') | Source/devtools/protocol.json » ('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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "config.h" 5 #include "config.h"
6 #include "core/inspector/PromiseTracker.h" 6 #include "core/inspector/PromiseTracker.h"
7 7
8 #include "bindings/core/v8/ScopedPersistent.h" 8 #include "bindings/core/v8/ScopedPersistent.h"
9 #include "bindings/core/v8/ScriptCallStackFactory.h" 9 #include "bindings/core/v8/ScriptCallStackFactory.h"
10 #include "bindings/core/v8/ScriptState.h" 10 #include "bindings/core/v8/ScriptState.h"
11 #include "wtf/PassOwnPtr.h" 11 #include "wtf/PassOwnPtr.h"
12 #include "wtf/WeakPtr.h" 12 #include "wtf/WeakPtr.h"
13 13
14 using blink::TypeBuilder::Array;
15 using blink::TypeBuilder::Console::CallFrame;
16 using blink::TypeBuilder::Debugger::PromiseDetails;
17
14 namespace blink { 18 namespace blink {
15 19
16 class PromiseTracker::PromiseData : public RefCounted<PromiseData> { 20 class PromiseTracker::PromiseData : public RefCounted<PromiseData> {
17 public: 21 public:
18 PromiseData(v8::Isolate* isolate, int promiseHash, v8::Handle<v8::Object> pr omise) 22 PromiseData(v8::Isolate* isolate, int promiseHash, int promiseId, v8::Handle <v8::Object> promise)
19 : m_promiseHash(promiseHash) 23 : m_promiseHash(promiseHash)
24 , m_promiseId(promiseId)
20 , m_promise(isolate, promise) 25 , m_promise(isolate, promise)
26 , m_parentPromiseId(0)
21 , m_status(0) 27 , m_status(0)
22 , m_weakPtrFactory(this) 28 , m_weakPtrFactory(this)
23 { 29 {
24 } 30 }
25 31
26 int promiseHash() const { return m_promiseHash; } 32 int promiseHash() const { return m_promiseHash; }
27 ScopedPersistent<v8::Object>& promise() { return m_promise; } 33 ScopedPersistent<v8::Object>& promise() { return m_promise; }
28 34
29 private: 35 private:
30 friend class PromiseTracker; 36 friend class PromiseTracker;
31 37
32 int m_promiseHash; 38 int m_promiseHash;
33 39 int m_promiseId;
34 ScopedPersistent<v8::Object> m_promise; 40 ScopedPersistent<v8::Object> m_promise;
35 ScriptCallFrame m_callFrame; 41 int m_parentPromiseId;
42 int m_status;
43 WeakPtrFactory<PromiseData> m_weakPtrFactory;
44 RefPtr<ScriptCallStack> m_callStack;
36 ScopedPersistent<v8::Object> m_parentPromise; 45 ScopedPersistent<v8::Object> m_parentPromise;
37 int m_status;
38
39 WeakPtrFactory<PromiseData> m_weakPtrFactory;
40 }; 46 };
41 47
42 static int indexOf(PromiseTracker::PromiseDataVector* vector, const ScopedPersis tent<v8::Object>& promise) 48 static int indexOf(PromiseTracker::PromiseDataVector* vector, const ScopedPersis tent<v8::Object>& promise)
43 { 49 {
44 for (size_t index = 0; index < vector->size(); ++index) { 50 for (size_t index = 0; index < vector->size(); ++index) {
45 if (vector->at(index)->promise() == promise) 51 if (vector->at(index)->promise() == promise)
46 return index; 52 return index;
47 } 53 }
48 return -1; 54 return -1;
49 } 55 }
(...skipping 26 matching lines...) Expand all
76 82
77 private: 83 private:
78 WeakPtr<PromiseTracker::PromiseData> m_data; 84 WeakPtr<PromiseTracker::PromiseData> m_data;
79 PromiseTracker::PromiseDataMap* m_promiseDataMap; 85 PromiseTracker::PromiseDataMap* m_promiseDataMap;
80 }; 86 };
81 87
82 } 88 }
83 89
84 PromiseTracker::PromiseTracker() 90 PromiseTracker::PromiseTracker()
85 : m_isEnabled(false) 91 : m_isEnabled(false)
92 , m_circularSequentialId(0)
86 { 93 {
87 } 94 }
88 95
89 PromiseTracker::~PromiseTracker() 96 PromiseTracker::~PromiseTracker()
90 { 97 {
91 } 98 }
92 99
93 void PromiseTracker::enable() 100 void PromiseTracker::setEnabled(bool enabled)
94 { 101 {
95 m_isEnabled = true; 102 m_isEnabled = enabled;
96 } 103 if (!enabled)
97 104 clear();
98 void PromiseTracker::disable()
99 {
100 m_isEnabled = false;
101 clear();
102 } 105 }
103 106
104 void PromiseTracker::clear() 107 void PromiseTracker::clear()
105 { 108 {
106 m_promiseDataMap.clear(); 109 m_promiseDataMap.clear();
107 } 110 }
108 111
109 void PromiseTracker::didReceiveV8PromiseEvent(ScriptState* scriptState, v8::Hand le<v8::Object> promise, v8::Handle<v8::Value> parentPromise, int status) 112 int PromiseTracker::circularSequentialId()
110 { 113 {
111 ASSERT(isEnabled()); 114 ++m_circularSequentialId;
115 if (m_circularSequentialId <= 0)
116 m_circularSequentialId = 1;
117 return m_circularSequentialId;
118 }
112 119
120 PassRefPtr<PromiseTracker::PromiseData> PromiseTracker::createPromiseDataIfNeede d(v8::Isolate* isolate, v8::Handle<v8::Object> promise)
121 {
113 int promiseHash = promise->GetIdentityHash(); 122 int promiseHash = promise->GetIdentityHash();
114 PromiseDataVector* vector; 123 PromiseDataVector* vector;
115 PromiseDataMap::iterator it = m_promiseDataMap.find(promiseHash); 124 PromiseDataMap::iterator it = m_promiseDataMap.find(promiseHash);
116 if (it != m_promiseDataMap.end()) 125 if (it != m_promiseDataMap.end())
117 vector = &it->value; 126 vector = &it->value;
118 else 127 else
119 vector = &m_promiseDataMap.add(promiseHash, PromiseDataVector()).storedV alue->value; 128 vector = &m_promiseDataMap.add(promiseHash, PromiseDataVector()).storedV alue->value;
120 129
121 v8::Isolate* isolate = scriptState->isolate();
122 RefPtr<PromiseData> data; 130 RefPtr<PromiseData> data;
123 int index = indexOf(vector, ScopedPersistent<v8::Object>(isolate, promise)); 131 int index = indexOf(vector, ScopedPersistent<v8::Object>(isolate, promise));
124 if (index == -1) { 132 if (index == -1) {
125 data = adoptRef(new PromiseData(isolate, promiseHash, promise)); 133 data = adoptRef(new PromiseData(isolate, promiseHash, circularSequential Id(), promise));
126 OwnPtr<PromiseDataWrapper> wrapper = adoptPtr(new PromiseDataWrapper(dat a->m_weakPtrFactory.createWeakPtr(), &m_promiseDataMap)); 134 OwnPtr<PromiseDataWrapper> wrapper = adoptPtr(new PromiseDataWrapper(dat a->m_weakPtrFactory.createWeakPtr(), &m_promiseDataMap));
127 data->m_promise.setWeak(wrapper.leakPtr(), &PromiseDataWrapper::didRemov ePromise); 135 data->m_promise.setWeak(wrapper.leakPtr(), &PromiseDataWrapper::didRemov ePromise);
128 vector->append(data); 136 vector->append(data);
129 } else { 137 } else {
130 data = vector->at(index); 138 data = vector->at(index);
131 } 139 }
132 140
133 if (!parentPromise.IsEmpty()) { 141 return data.release();
134 ASSERT(parentPromise->IsObject()); 142 }
135 data->m_parentPromise.set(isolate, parentPromise->ToObject()); 143
144 void PromiseTracker::didReceiveV8PromiseEvent(ScriptState* scriptState, v8::Hand le<v8::Object> promise, v8::Handle<v8::Value> parentPromise, int status)
145 {
146 ASSERT(isEnabled());
147
148 v8::Isolate* isolate = scriptState->isolate();
149 RefPtr<PromiseData> data = createPromiseDataIfNeeded(isolate, promise);
150 if (!parentPromise.IsEmpty() && parentPromise->IsObject()) {
151 v8::Handle<v8::Object> handle = parentPromise->ToObject();
152 RefPtr<PromiseData> parentData = createPromiseDataIfNeeded(isolate, hand le);
153 data->m_parentPromiseId = parentData->m_promiseId;
154 data->m_parentPromise.set(isolate, handle);
136 } else { 155 } else {
137 data->m_status = status; 156 data->m_status = status;
138 if (!status) { 157 if (!status && !data->m_callStack) {
139 v8::Handle<v8::StackTrace> stackTrace(v8::StackTrace::CurrentStackTr ace(isolate, 1)); 158 v8::Handle<v8::StackTrace> stackTrace(v8::StackTrace::CurrentStackTr ace(isolate, 1));
140 RefPtrWillBeRawPtr<ScriptCallStack> stack = createScriptCallStack(st ackTrace, 1, isolate); 159 RefPtrWillBeRawPtr<ScriptCallStack> stack = createScriptCallStack(st ackTrace, 1, isolate);
141 if (stack->size()) 160 if (stack->size())
142 data->m_callFrame = stack->at(0); 161 data->m_callStack = stack;
143 } 162 }
144 } 163 }
145 } 164 }
146 165
166 PassRefPtr<Array<PromiseDetails> > PromiseTracker::promises()
167 {
168 ASSERT(isEnabled());
169
170 RefPtr<Array<PromiseDetails> > result = Array<PromiseDetails>::create();
171 for (PromiseDataMap::iterator it = m_promiseDataMap.begin(); it != m_promise DataMap.end(); ++it) {
172 PromiseDataVector* vector = &it->value;
173 for (size_t index = 0; index < vector->size(); ++index) {
174 RefPtr<PromiseData> data = vector->at(index);
175 PromiseDetails::Status::Enum status;
176 if (!data->m_status)
177 status = PromiseDetails::Status::Pending;
178 else if (data->m_status == 1)
179 status = PromiseDetails::Status::Resolved;
180 else
181 status = PromiseDetails::Status::Rejected;
182 RefPtr<PromiseDetails> promiseDetails = PromiseDetails::create()
183 .setId(data->m_promiseId)
184 .setStatus(status);
185 if (data->m_parentPromiseId)
186 promiseDetails->setParentId(data->m_parentPromiseId);
187 if (data->m_callStack)
188 promiseDetails->setCallFrame(data->m_callStack->at(0).buildInspe ctorObject());
189 result->addItem(promiseDetails);
190 }
191 }
192
193 return result.release();
194 }
195
147 } // namespace blink 196 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/inspector/PromiseTracker.h ('k') | Source/devtools/protocol.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698