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

Side by Side Diff: cc/debug/benchmark_instrumentation.cc

Issue 1717283003: tracing: Make ConvertableToTraceFormat move-only scoped_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/trace_event/trace_event.h" 5 #include "base/trace_event/trace_event.h"
6 #include "cc/debug/benchmark_instrumentation.h" 6 #include "cc/debug/benchmark_instrumentation.h"
7 7
8 namespace cc { 8 namespace cc {
9 namespace benchmark_instrumentation { 9 namespace benchmark_instrumentation {
10 10
11 // Please do not change the trace events in this file without updating 11 // Please do not change the trace events in this file without updating
12 // tools/perf/measurements/rendering_stats.py accordingly. 12 // tools/perf/measurements/rendering_stats.py accordingly.
13 // The benchmarks search for events and their arguments by name. 13 // The benchmarks search for events and their arguments by name.
14 14
15 void IssueImplThreadRenderingStatsEvent(const RenderingStats& stats) { 15 void IssueImplThreadRenderingStatsEvent(const RenderingStats& stats) {
16 TRACE_EVENT_INSTANT1("benchmark", 16 TRACE_EVENT_INSTANT1("benchmark",
17 "BenchmarkInstrumentation::ImplThreadRenderingStats", 17 "BenchmarkInstrumentation::ImplThreadRenderingStats",
18 TRACE_EVENT_SCOPE_THREAD, 18 TRACE_EVENT_SCOPE_THREAD,
19 "data", stats.AsTraceableData()); 19 "data", stats.AsTraceableData());
20 } 20 }
21 21
22 void IssueDisplayRenderingStatsEvent() { 22 void IssueDisplayRenderingStatsEvent() {
23 scoped_refptr<base::trace_event::TracedValue> record_data = 23 auto record_data = make_scoped_ptr(new base::trace_event::TracedValue());
danakj 2016/02/25 23:10:07 current guidance is "don't spell smart pointers as
Primiano Tucci (use gerrit) 2016/02/26 07:01:04 Ok changed all the places in the CL where I did au
24 new base::trace_event::TracedValue();
25 record_data->SetInteger("frame_count", 1); 24 record_data->SetInteger("frame_count", 1);
26 TRACE_EVENT_INSTANT1( 25 TRACE_EVENT_INSTANT1(
27 "benchmark", 26 "benchmark", "BenchmarkInstrumentation::DisplayRenderingStats",
28 "BenchmarkInstrumentation::DisplayRenderingStats", 27 TRACE_EVENT_SCOPE_THREAD, "data", std::move(record_data));
29 TRACE_EVENT_SCOPE_THREAD,
30 "data",
31 scoped_refptr<base::trace_event::ConvertableToTraceFormat>(record_data));
32 } 28 }
33 29
34 } // namespace benchmark_instrumentation 30 } // namespace benchmark_instrumentation
35 } // namespace cc 31 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698