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

Side by Side Diff: content/browser/tracing/power_tracing_agent.cc

Issue 1468173003: [Tracing Clock Sync] Add TracingAgent interface in Chrome (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/lazy_instance.h" 5 #include "base/lazy_instance.h"
6 #include "base/memory/singleton.h" 6 #include "base/memory/singleton.h"
7 #include "base/trace_event/trace_event_impl.h" 7 #include "base/trace_event/trace_event_impl.h"
8 #include "content/browser/tracing/battor_power_trace_provider.h" 8 #include "content/browser/tracing/battor_power_trace_provider.h"
9 #include "content/browser/tracing/power_tracing_agent.h" 9 #include "content/browser/tracing/power_tracing_agent.h"
10 #include "content/public/browser/browser_thread.h" 10 #include "content/public/browser/browser_thread.h"
11 11
12 namespace content { 12 namespace content {
13 13
14 // static
15 PowerTracingAgent* PowerTracingAgent::GetInstance() {
16 return base::Singleton<PowerTracingAgent>::get();
17 }
18
14 PowerTracingAgent::PowerTracingAgent() : is_tracing_(false) { 19 PowerTracingAgent::PowerTracingAgent() : is_tracing_(false) {
15 battor_trace_provider_.reset(new BattorPowerTraceProvider()); 20 battor_trace_provider_.reset(new BattorPowerTraceProvider());
16 } 21 }
17 22
18 PowerTracingAgent::~PowerTracingAgent() {} 23 PowerTracingAgent::~PowerTracingAgent() {}
19 24
20 bool PowerTracingAgent::StartTracing() { 25 std::string PowerTracingAgent::GetTracingAgentName() {
26 return kPowerTracingAgentName;
27 }
28
29 bool PowerTracingAgent::StartAgentTracing(
30 const base::trace_event::TraceConfig& trace_config) {
21 // Tracing session already in progress. 31 // Tracing session already in progress.
22 if (is_tracing_) 32 if (is_tracing_)
23 return false; 33 return false;
24 34
25 // TODO(prabhur) Start tracing probably needs to be done in a 35 // TODO(prabhur) Start tracing probably needs to be done in a
26 // separate thread since it involves talking to the h/w. 36 // separate thread since it involves talking to the h/w.
27 // Revisit once battor h/w communication is enabled. 37 // Revisit once battor h/w communication is enabled.
28 is_tracing_ = battor_trace_provider_->StartTracing(); 38 is_tracing_ = battor_trace_provider_->StartTracing();
29 return is_tracing_; 39 return is_tracing_;
30 } 40 }
31 41
32 void PowerTracingAgent::StopTracing(const OutputCallback& callback) { 42 void PowerTracingAgent::StopAgentTracing(
43 const StopAgentTracingCallback& callback) {
33 // No tracing session in progress. 44 // No tracing session in progress.
34 if (!is_tracing_) 45 if (!is_tracing_)
35 return; 46 return;
36 47
37 // Stop tracing & collect logs. 48 // Stop tracing & collect logs.
38 OutputCallback on_stop_power_tracing_done_callback = base::Bind( 49 StopAgentTracingCallback on_stop_power_tracing_done_callback = base::Bind(
39 &PowerTracingAgent::OnStopTracingDone, base::Unretained(this), callback); 50 &PowerTracingAgent::OnStopTracingDone, base::Unretained(this), callback);
40 BrowserThread::PostTask( 51 BrowserThread::PostTask(
41 BrowserThread::UI, FROM_HERE, 52 BrowserThread::UI, FROM_HERE,
42 base::Bind(&PowerTracingAgent::FlushOnThread, base::Unretained(this), 53 base::Bind(&PowerTracingAgent::FlushOnThread, base::Unretained(this),
43 on_stop_power_tracing_done_callback)); 54 on_stop_power_tracing_done_callback));
44 } 55 }
45 56
46 void PowerTracingAgent::OnStopTracingDone( 57 void PowerTracingAgent::OnStopTracingDone(
47 const OutputCallback& callback, 58 const StopAgentTracingCallback& callback,
48 const scoped_refptr<base::RefCountedString>& result) { 59 const scoped_refptr<base::RefCountedString>& result) {
49 is_tracing_ = false; 60 is_tracing_ = false;
50 // Pass the serialized events. 61 // Pass the serialized events.
51 callback.Run(result); 62 callback.Run(result);
52 } 63 }
53 64
54 // static 65 void PowerTracingAgent::FlushOnThread(
55 PowerTracingAgent* PowerTracingAgent::GetInstance() { 66 const StopAgentTracingCallback& callback) {
56 return base::Singleton<PowerTracingAgent>::get();
57 }
58
59 void PowerTracingAgent::FlushOnThread(const OutputCallback& callback) {
60 // Pass the result to the UI Thread. 67 // Pass the result to the UI Thread.
61 68
62 // TODO(prabhur) StopTracing & GetLog need to be called on a 69 // TODO(prabhur) StopTracing & GetLog need to be called on a
63 // separate thread depending on how BattorPowerTraceProvider is implemented. 70 // separate thread depending on how BattorPowerTraceProvider is implemented.
64 battor_trace_provider_->StopTracing(); 71 battor_trace_provider_->StopTracing();
65 std::string battor_logs; 72 std::string battor_logs;
66 battor_trace_provider_->GetLog(&battor_logs); 73 battor_trace_provider_->GetLog(&battor_logs);
67 scoped_refptr<base::RefCountedString> result = 74 scoped_refptr<base::RefCountedString> result =
68 base::RefCountedString::TakeString(&battor_logs); 75 base::RefCountedString::TakeString(&battor_logs);
69 BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, 76 BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
70 base::Bind(callback, result)); 77 base::Bind(callback, result));
71 } 78 }
72 79
80 bool PowerTracingAgent::SupportsExplicitClockSync() {
81 return false;
82 }
83
84 void PowerTracingAgent::RecordClockSyncMarker(
85 scoped_ptr<base::DictionaryValue> marker) {
86 DCHECK(SupportsExplicitClockSync());
87 }
88
89 void PowerTracingAgent::IssueClockSyncMarker() {
90 DCHECK(SupportsExplicitClockSync());
91 }
92
73 } // namespace content 93 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698