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

Unified Diff: content/browser/renderer_host/render_widget_host_impl.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: content/browser/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index e81f0f3483598c5f739428f7288904567e3350cf..cdee694d677237668fc637e47de66d341d9b1751 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1636,9 +1636,11 @@ void RenderWidgetHostImpl::OnUpdateRect(
GetProcess()->ReceivedBadMessage();
} else {
UNSHIPPED_TRACE_EVENT_INSTANT2("test_latency", "UpdateRect",
+ TRACE_EVENT_SCOPE_THREAD,
"x+y", params.bitmap_rect.x() + params.bitmap_rect.y(),
"color", 0xffffff & *static_cast<uint32*>(dib->memory()));
UNSHIPPED_TRACE_EVENT_INSTANT1("test_latency", "UpdateRectWidth",
+ TRACE_EVENT_SCOPE_THREAD,
"width", params.bitmap_rect.width());
// Scroll the backing store.
@@ -1751,6 +1753,7 @@ void RenderWidgetHostImpl::DidUpdateBackingStore(
delta = now - paint_start;
UMA_HISTOGRAM_TIMES("MPArch.RWH_TotalPaintTime", delta);
UNSHIPPED_TRACE_EVENT_INSTANT1("test_latency", "UpdateRectComplete",
+ TRACE_EVENT_SCOPE_THREAD,
"x+y", params.bitmap_rect.x() + params.bitmap_rect.y());
}
@@ -1837,7 +1840,8 @@ void RenderWidgetHostImpl::TickActiveSmoothScrollGesture() {
TRACE_EVENT0("input", "RenderWidgetHostImpl::TickActiveSmoothScrollGesture");
tick_active_smooth_scroll_gestures_task_posted_ = false;
if (active_smooth_scroll_gestures_.empty()) {
- TRACE_EVENT_INSTANT0("input", "EarlyOut_NoActiveScrollGesture");
+ TRACE_EVENT_INSTANT0("input", "EarlyOut_NoActiveScrollGesture",
+ TRACE_EVENT_SCOPE_THREAD);
return;
}
@@ -1849,7 +1853,7 @@ void RenderWidgetHostImpl::TickActiveSmoothScrollGesture() {
now;
if (time_until_next_ideal_interval.InMilliseconds() > 0) {
TRACE_EVENT_INSTANT1(
- "input", "EarlyOut_TickedTooRecently",
+ "input", "EarlyOut_TickedTooRecently", TRACE_EVENT_SCOPE_THREAD,
"delay", time_until_next_ideal_interval.InMilliseconds());
// Post a task.
tick_active_smooth_scroll_gestures_task_posted_ = true;
@@ -1890,7 +1894,7 @@ void RenderWidgetHostImpl::TickActiveSmoothScrollGesture() {
if (!in_process_event_types_.empty())
return;
- TRACE_EVENT_INSTANT1("input", "PostTickTask",
+ TRACE_EVENT_INSTANT1("input", "PostTickTask", TRACE_EVENT_SCOPE_THREAD,
"delay", preferred_interval.InMilliseconds());
tick_active_smooth_scroll_gestures_task_posted_ = true;
MessageLoop::current()->PostDelayedTask(

Powered by Google App Engine
This is Rietveld 408576698