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

Issue 23556003: Implement about:tracing UI for the sampling profiler (Chromium part)

Created:
7 years, 3 months ago by haraken
Modified:
6 years, 6 months ago
Reviewers:
scottmg
CC:
chromium-reviews, darin-cc_chromium.org, dsinclair+watch_chromium.org, erikwright+watch_chromium.org, jam, joi+watch-content_chromium.org
Visibility:
Public.

Description

Implement about:tracing UI for the sampling profiler (Chromium part) The design document of the sampling profiler is https://docs.google.com/a/google.com/document/d/1j39sbA9ECTwFFlxbhSD1EvJnGfYeS5EUJ05yxuku6VY/edit# The trace-viewer side CL is here: https://codereview.appspot.com/13273043/ BUG=241743 TEST=base_unittests::TraceEventTestFixture.TraceContinuousSampling

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+635 lines, -49 lines) Patch
M base/debug/trace_event.h View 2 chunks +6 lines, -4 lines 0 comments Download
M base/debug/trace_event_impl.h View 1 2 7 chunks +21 lines, -2 lines 0 comments Download
M base/debug/trace_event_impl.cc View 1 27 chunks +170 lines, -28 lines 0 comments Download
M base/debug/trace_event_unittest.cc View 1 2 chunks +56 lines, -0 lines 0 comments Download
M components/tracing/child_trace_message_filter.h View 2 chunks +5 lines, -0 lines 0 comments Download
M components/tracing/child_trace_message_filter.cc View 1 3 chunks +43 lines, -0 lines 0 comments Download
M components/tracing/tracing_messages.h View 1 3 chunks +19 lines, -1 line 0 comments Download
M content/browser/tracing/trace_controller_impl.h View 3 chunks +41 lines, -9 lines 0 comments Download
M content/browser/tracing/trace_controller_impl.cc View 7 chunks +125 lines, -4 lines 0 comments Download
M content/browser/tracing/trace_message_filter.h View 2 chunks +7 lines, -0 lines 0 comments Download
M content/browser/tracing/trace_message_filter.cc View 5 chunks +45 lines, -0 lines 0 comments Download
M content/browser/tracing/tracing_ui.cc View 7 chunks +83 lines, -0 lines 0 comments Download
M content/public/browser/trace_controller.h View 1 chunk +5 lines, -0 lines 0 comments Download
M content/public/browser/trace_subscriber.h View 1 chunk +5 lines, -0 lines 0 comments Download
M content/renderer/render_view_impl.cc View 1 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
haraken
7 years, 3 months ago (2013-09-03 06:47:51 UTC) #1
Now I added tests. PTAL.

Powered by Google App Engine
This is Rietveld 408576698