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

Unified Diff: cc/thread_proxy.cc

Issue 12252058: Add a |scope| argument to TRACE_EVENT_INSTANT* and require its presence. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use flags to record scope Created 7 years, 10 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
Index: cc/thread_proxy.cc
diff --git a/cc/thread_proxy.cc b/cc/thread_proxy.cc
index ddc61c07dad0ff2748ad506c0eb6b91a5012acb7..c988a4d978f204d480c684692306d1618c4902f7 100644
--- a/cc/thread_proxy.cc
+++ b/cc/thread_proxy.cc
@@ -760,7 +760,7 @@ void ThreadProxy::scheduledActionCommit()
{
// For some layer types in impl-side painting, the commit is held until
// the pending tree is activated.
- TRACE_EVENT_INSTANT0("cc", "HoldCommit");
+ TRACE_EVENT_INSTANT0("cc", "HoldCommit", TRACE_EVENT_SCOPE_THREAD);
m_completionEventForCommitHeldOnTreeActivation = m_commitCompletionEventOnImplThread;
m_commitCompletionEventOnImplThread = 0;
}
@@ -842,7 +842,8 @@ ScheduledActionDrawAndSwapResult ThreadProxy::scheduledActionDrawAndSwapInternal
// Check for tree activation.
if (m_completionEventForCommitHeldOnTreeActivation && !m_layerTreeHostImpl->pendingTree())
{
- TRACE_EVENT_INSTANT0("cc", "ReleaseCommitbyActivation");
+ TRACE_EVENT_INSTANT0("cc", "ReleaseCommitbyActivation",
+ TRACE_EVENT_SCOPE_THREAD);
DCHECK(m_layerTreeHostImpl->settings().implSidePainting);
m_completionEventForCommitHeldOnTreeActivation->signal();
m_completionEventForCommitHeldOnTreeActivation = 0;

Powered by Google App Engine
This is Rietveld 408576698