Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 |
| OLD | NEW |