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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp

Issue 2427493002: [wrapper-tracing] Clear reporter after lazy cleanup (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp
index 86ed1b1de255d4768a18efcef3be8eb5e46c700b..1ed50fa5db2fb9b13bc4c1b828478dc8e89c5c38 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptWrappableVisitor.cpp
@@ -81,9 +81,9 @@ void ScriptWrappableVisitor::performCleanup() {
m_headersToUnmark.clear();
m_markingDeque.clear();
m_verifierDeque.clear();
+ m_reporter = nullptr;
m_shouldCleanup = false;
m_tracingInProgress = false;
- m_reporter = nullptr;
}
void ScriptWrappableVisitor::scheduleIdleLazyCleanup() {
@@ -137,6 +137,7 @@ void ScriptWrappableVisitor::performLazyCleanup(double deadlineSeconds) {
CHECK(m_headersToUnmark.isEmpty());
m_markingDeque.clear();
m_verifierDeque.clear();
+ m_reporter = nullptr;
m_shouldCleanup = false;
m_tracingInProgress = false;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698