DevTools: use async operation id instead of AsyncCallChain

Source/core/inspector/InspectorDebuggerAgent.cpp

 /*
  * Copyright (C) 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 112 matching lines @@
     , m_pausingOnNativeEvent(false)
     , m_inAsyncOperationForStepInto(false)
     , m_listener(nullptr)
     , m_skippedStepFrameCount(0)
     , m_recursionLevelForStepOut(0)
     , m_recursionLevelForStepFrame(0)
     , m_skipAllPauses(false)
     , m_skipContentScripts(false)
     , m_cachedSkipStackGeneration(0)
     , m_promiseTracker(PromiseTracker::create())
+    , m_nextAsyncOperationId(1)
     , m_maxAsyncCallStackDepth(0)
     , m_currentAsyncCallChain(nullptr)
     , m_nestedAsyncCallCount(0)
     , m_performingAsyncStepIn(false)
 {
     m_v8AsyncCallTracker = V8AsyncCallTracker::create(this);
 }
 
 InspectorDebuggerAgent::~InspectorDebuggerAgent()
 {
@@ skipping 933 matching lines @@
     }
     InjectedScript injectedScript = m_injectedScriptManager->injectedScriptFor(value.scriptState());
     promise = injectedScript.wrapObject(value, objectGroup ? *objectGroup : "");
 }
 
 const AsyncCallChain* InspectorDebuggerAgent::currentAsyncCallChain() const
 {
     return m_currentAsyncCallChain.get();
 }
 
-PassRefPtrWillBeRawPtr<AsyncCallChain> InspectorDebuggerAgent::traceAsyncOperationStarting(const String& description)
+int InspectorDebuggerAgent::traceAsyncOperationStarting(const String& description)
 {
     ScriptValue callFrames = scriptDebugServer().currentCallFramesForAsyncStack();
     if (callFrames.isEmpty()) {
         if (!m_currentAsyncCallChain)
-            return nullptr;
-        RefPtrWillBeRawPtr<AsyncCallChain> chain = AsyncCallChain::create(nullptr, m_currentAsyncCallChain.get(), m_maxAsyncCallStackDepth);
-        didCreateAsyncCallChain(chain.get());
-        return chain.release(); // Propagate async call stack chain.
+            return 0;
+        return createAsyncCallChain(nullptr);
     }
-    RefPtrWillBeRawPtr<AsyncCallChain> chain = AsyncCallChain::create(adoptRefWillBeNoop(new AsyncCallStack(description, callFrames)), m_currentAsyncCallChain.get(), m_maxAsyncCallStackDepth);
+    return createAsyncCallChain(adoptRefWillBeNoop(new AsyncCallStack(description, callFrames)));
+}
+
+int InspectorDebuggerAgent::createAsyncCallChain(PassRefPtrWillBeRawPtr<AsyncCallStack> callStack)

[aandrey, 2015/01/20 14:33:36]

inline this method?

[yurys, 2015/01/20 17:15:38]

Done.

+{
+    RefPtrWillBeRawPtr<AsyncCallChain> chain = AsyncCallChain::create(callStack, m_currentAsyncCallChain.get(), m_maxAsyncCallStackDepth);
+    while (!m_nextAsyncOperationId || m_asyncOperations.contains(m_nextAsyncOperationId))
+        ++m_nextAsyncOperationId;
+    int id = m_nextAsyncOperationId++;

[aandrey, 2015/01/20 14:33:36]

should be circular, never negative

[yurys, 2015/01/20 17:15:38]

Done.

+    m_asyncOperations.set(id, chain);
     didCreateAsyncCallChain(chain.get());
-    return chain.release();
+    return id;
 }
 
 void InspectorDebuggerAgent::didCreateAsyncCallChain(AsyncCallChain* chain)

[aandrey, 2015/01/20 14:33:36]

this can be inlined too

[yurys, 2015/01/20 17:15:38]

Done.

 {
     if (!m_performingAsyncStepIn)
         return;
     if (m_inAsyncOperationForStepInto || m_asyncOperationsForStepInto.isEmpty())
         m_asyncOperationsForStepInto.add(chain);
 }
 
 void InspectorDebuggerAgent::traceAsyncCallbackCompleted()
 {
     if (!m_nestedAsyncCallCount)
         return;
     ASSERT(m_currentAsyncCallChain);
     --m_nestedAsyncCallCount;
     if (!m_nestedAsyncCallCount) {
         m_currentAsyncCallChain.clear();
         if (!m_performingAsyncStepIn)
             return;
         if (!m_inAsyncOperationForStepInto)
             return;
         m_inAsyncOperationForStepInto = false;
         m_scheduledDebuggerStep = NoStep;
         scriptDebugServer().setPauseOnNextStatement(false);
         if (m_asyncOperationsForStepInto.isEmpty())
             clearStepIntoAsync();
     }
 }
 
-void InspectorDebuggerAgent::traceAsyncCallbackStarting(v8::Isolate* isolate, PassRefPtrWillBeRawPtr<AsyncCallChain> chain)
+void InspectorDebuggerAgent::traceAsyncCallbackStarting(v8::Isolate* isolate, int operationId)
 {
+    AsyncCallChain* chain = m_asyncOperations.get(operationId);

[aandrey, 2015/01/20 14:33:36]

ASSERT(operationId > 0);

[yurys, 2015/01/20 17:15:38]

Done.

     int recursionLevel = V8RecursionScope::recursionLevel(isolate);
     if (chain && (!recursionLevel || (recursionLevel == 1 && Microtask::performingCheckpoint(isolate)))) {
         // Current AsyncCallChain corresponds to the bottommost JS call frame.
         m_currentAsyncCallChain = chain;
         m_nestedAsyncCallCount = 1;
         if (!m_performingAsyncStepIn)
             return;
         if (!m_asyncOperationsForStepInto.contains(m_currentAsyncCallChain.get()))
             return;
         m_inAsyncOperationForStepInto = true;
         m_scheduledDebuggerStep = StepInto;
         m_skippedStepFrameCount = 0;
         m_recursionLevelForStepFrame = 0;
         scriptDebugServer().setPauseOnNextStatement(true);
     } else {
         if (m_currentAsyncCallChain)
             ++m_nestedAsyncCallCount;
     }
 }
 
-void InspectorDebuggerAgent::traceAsyncOperationCompleted(AsyncCallChain* chain)
+void InspectorDebuggerAgent::traceAsyncOperationCompleted(int operationId)
 {
+    RefPtrWillBeRawPtr<AsyncCallChain> chain = m_asyncOperations.take(operationId);

[aandrey, 2015/01/20 14:33:36]

ditto

[yurys, 2015/01/20 17:15:38]

Done.

     if (!m_performingAsyncStepIn)
         return;
     m_asyncOperationsForStepInto.remove(chain);

[aandrey, 2015/01/20 14:33:36]

can it be HashSet<int> now?

[yurys, 2015/01/20 17:15:38]

Done.

     if (!m_inAsyncOperationForStepInto && m_asyncOperationsForStepInto.isEmpty())
         clearStepIntoAsync();
 }
 
 void InspectorDebuggerAgent::resetAsyncCallTracker()
 {
     m_currentAsyncCallChain.clear();
     m_nestedAsyncCallCount = 0;
     for (auto& listener: m_asyncCallTrackingListeners)
         listener->resetAsyncCallChains();
@@ skipping 392 matching lines @@
     visitor->trace(m_v8AsyncCallTracker);
     visitor->trace(m_promiseTracker);
     visitor->trace(m_asyncOperationsForStepInto);
     visitor->trace(m_currentAsyncCallChain);
     visitor->trace(m_asyncCallTrackingListeners);
 #endif
     InspectorBaseAgent::trace(visitor);
 }
 
 } // namespace blink