 Chromium Code Reviews
 Chromium Code Reviews Issue 177773002:
  Support Promises instrumentation on backend.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master
    
  
    Issue 177773002:
  Support Promises instrumentation on backend.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: Source/core/inspector/PromiseOfficer.cpp | 
| diff --git a/Source/core/inspector/PromiseOfficer.cpp b/Source/core/inspector/PromiseOfficer.cpp | 
| new file mode 100644 | 
| index 0000000000000000000000000000000000000000..94204d7d920b9083be7940398ba3096c9559dc62 | 
| --- /dev/null | 
| +++ b/Source/core/inspector/PromiseOfficer.cpp | 
| @@ -0,0 +1,129 @@ | 
| +/* | 
| + * Copyright (C) 2014 Google Inc. All rights reserved. | 
| + * | 
| + * Redistribution and use in source and binary forms, with or without | 
| + * modification, are permitted provided that the following conditions are | 
| + * met: | 
| + * | 
| + * * Redistributions of source code must retain the above copyright | 
| + * notice, this list of conditions and the following disclaimer. | 
| + * * Redistributions in binary form must reproduce the above | 
| + * copyright notice, this list of conditions and the following disclaimer | 
| + * in the documentation and/or other materials provided with the | 
| + * distribution. | 
| + * * Neither the name of Google Inc. nor the names of its | 
| + * contributors may be used to endorse or promote products derived from | 
| + * this software without specific prior written permission. | 
| + * | 
| + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS | 
| + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT | 
| + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR | 
| + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT | 
| + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, | 
| + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT | 
| + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, | 
| + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY | 
| + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 
| + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 
| + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 
| + */ | 
| + | 
| +#include "config.h" | 
| + | 
| +#include "bindings/v8/ScriptCallStackFactory.h" | 
| +#include "bindings/v8/V8Binding.h" | 
| +#include "core/inspector/PromiseOfficer.h" | 
| +#include "wtf/CurrentTime.h" | 
| + | 
| +namespace WebCore { | 
| + | 
| +PromiseOfficer::PromiseData::PromiseData(v8::Handle<v8::Object> promise, v8::Handle<v8::Object> parentPromise, v8::Handle<v8::Value> result, V8PromiseCustom::PromiseState state, double timestamp, v8::Isolate* isolate) | 
| + : m_isolate(isolate) | 
| + , m_promise(m_isolate, promise) | 
| + , m_parentPromise(m_isolate, parentPromise) | 
| + , m_result(m_isolate, result) | 
| + , m_state(state) | 
| + , m_timeOnCreate(timestamp) | 
| + , m_timeOnSettle(-1) | 
| + , m_callStackOnCreate(createScriptCallStack(ScriptCallStack::maxCallStackSizeToCapture, true)) | 
| + , m_callStackOnSettle(RefPtr<ScriptCallStack>()) | 
| 
yurys
2014/03/05 15:23:35
compiler will generate this automatically
 
Alexandra Mikhaylova
2014/03/07 11:11:44
Done.
 | 
| +{ | 
| +} | 
| + | 
| +void PromiseOfficer::PromiseData::didSettlePromise(double timestamp) | 
| +{ | 
| + m_timeOnSettle = timestamp; | 
| + m_callStackOnSettle = createScriptCallStack(ScriptCallStack::maxCallStackSizeToCapture, true); | 
| +} | 
| + | 
| +void PromiseOfficer::enable() | 
| +{ | 
| + m_isEnabled = true; | 
| +} | 
| + | 
| +void PromiseOfficer::disable() | 
| +{ | 
| + m_isEnabled = false; | 
| + clear(); | 
| +} | 
| + | 
| +void PromiseOfficer::clear() | 
| +{ | 
| + m_promiseDataMap.clear(); | 
| +} | 
| + | 
| +void PromiseOfficer::didCreatePromise(ExecutionContext* context, v8::Handle<v8::Object> promise) | 
| +{ | 
| + ASSERT(isEnabled()); | 
| + | 
| + double timestamp = currentTimeMS(); | 
| + | 
| + int promiseHash = promise->GetIdentityHash(); | 
| + PromiseDataVector* vector; | 
| + PromiseDataMap::iterator it = m_promiseDataMap.find(promiseHash); | 
| 
yurys
2014/03/05 15:23:35
Why not use a HashMap (with appropriate HashTraits
 
Alexandra Mikhaylova
2014/03/07 11:11:44
Thanks, fixed it.
 | 
| + if (it != m_promiseDataMap.end()) | 
| + vector = &it->value; | 
| + else | 
| + vector = &m_promiseDataMap.add(promiseHash, PromiseDataVector()).storedValue->value; | 
| + v8::Isolate* isolate = toIsolate(context); | 
| + vector->append(adoptRef(new PromiseData(promise, v8::Handle<v8::Object>(), v8::Undefined(isolate), V8PromiseCustom::Pending, timestamp, isolate))); | 
| +} | 
| + | 
| +void PromiseOfficer::didUpdatePromiseParent(v8::Handle<v8::Object> promise, v8::Handle<v8::Object> parentPromise) | 
| +{ | 
| + ASSERT(isEnabled()); | 
| + | 
| + int promiseHash = promise->GetIdentityHash(); | 
| + PromiseDataMap::iterator it = m_promiseDataMap.find(promiseHash); | 
| + PromiseDataVector& vector = it->value; | 
| + for (size_t index = 0; index < vector.size(); ++index) { | 
| + RefPtr<PromiseData> data = vector[index]; | 
| + if (data->m_promise == promise) { | 
| + data->m_parentPromise.set(data->m_isolate, parentPromise); | 
| + break; | 
| + } | 
| + } | 
| +} | 
| + | 
| +void PromiseOfficer::didUpdatePromiseState(v8::Handle<v8::Object> promise, V8PromiseCustom::PromiseState state, v8::Handle<v8::Value> result) | 
| +{ | 
| + ASSERT(isEnabled()); | 
| + | 
| + double timestamp = currentTimeMS(); | 
| + | 
| + int promiseHash = promise->GetIdentityHash(); | 
| + PromiseDataMap::iterator it = m_promiseDataMap.find(promiseHash); | 
| + PromiseDataVector& vector = it->value; | 
| + for (size_t index = 0; index < vector.size(); ++index) { | 
| + RefPtr<PromiseData> data = vector[index]; | 
| + if (data->m_promise == promise) { | 
| + data->m_state = state; | 
| + data->m_result.set(data->m_isolate, result); | 
| + if (state == V8PromiseCustom::Fulfilled || state == V8PromiseCustom::Rejected) | 
| + data->didSettlePromise(timestamp); | 
| + break; | 
| + } | 
| + } | 
| +} | 
| + | 
| +} // namespace WebCore |