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

Issue 122053002: Revert 242671 "Implement sampling profiler (chromium side change)" (Closed)

Created:
6 years, 12 months ago by haraken
Modified:
6 years, 12 months ago
Reviewers:
haraken
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 242671 "Implement sampling profiler (chromium side change)" > Implement sampling profiler (chromium side change) > > Trace-viewer side change is here: https://codereview.appspot.com/39310043 > > Screenshot of the profiler: http://haraken.info/null/sampling_tracing.png > > Design document: https://docs.google.com/a/google.com/document/d/1j39sbA9ECTwFFlxbhSD1EvJnGfYeS5EUJ05yxuku6VY/edit > > TBR=phajdan.jr > NOTRY=true > > Review URL: https://codereview.chromium.org/109933006 TBR=haraken@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242679

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+64 lines, -226 lines) Patch
M trunk/src/base/debug/trace_event.h View 5 chunks +5 lines, -9 lines 0 comments Download
M trunk/src/base/debug/trace_event_android.cc View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/base/debug/trace_event_impl.h View 6 chunks +12 lines, -19 lines 0 comments Download
M trunk/src/base/debug/trace_event_impl.cc View 14 chunks +12 lines, -26 lines 0 comments Download
M trunk/src/base/debug/trace_event_unittest.cc View 36 chunks +20 lines, -72 lines 0 comments Download
M trunk/src/base/test/trace_event_analyzer_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/chrome/browser/lifetime/application_lifetime.cc View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/components/tracing/child_trace_message_filter.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M trunk/src/content/app/content_main_runner.cc View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/content/browser/media/webrtc_browsertest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/content/browser/tracing/tracing_controller_impl.cc View 4 chunks +3 lines, -11 lines 0 comments Download
M trunk/src/content/browser/tracing/tracing_ui.cc View 6 chunks +12 lines, -81 lines 0 comments Download
M trunk/src/content/common/gpu/client/gl_helper_unittests.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
haraken
6 years, 12 months ago (2013-12-28 08:21:45 UTC) #1
haraken
6 years, 12 months ago (2013-12-28 08:22:10 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r242679.

Powered by Google App Engine
This is Rietveld 408576698