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

Unified Diff: content/browser/tracing/tracing_controller_impl.cc

Issue 2400163003: arc: enable Android tracing in verified-boot mode (Closed)
Patch Set: address comments Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/tracing/tracing_controller_impl.cc
diff --git a/content/browser/tracing/tracing_controller_impl.cc b/content/browser/tracing/tracing_controller_impl.cc
index 9e49c0bec7c2d3a51070f2885887dff9293be262..95a387fd4e701d7bf6581bd52ada382f723c51ee 100644
--- a/content/browser/tracing/tracing_controller_impl.cc
+++ b/content/browser/tracing/tracing_controller_impl.cc
@@ -672,10 +672,11 @@ void TracingControllerImpl::OnEndAgentTracingAcked(
// The Windows kernel events are kept into a JSON format stored as string
// and must not be escaped.
trace_data_sink_->AddAgentTrace(events_label, events_str_ptr->data());
- } else if (agent_name != kArcTracingAgentName) {
- // ARC trace data is obtained via systrace. Ignore the empty data.
- // Quote other trace data as JSON strings and merge them into
- // |trace_data_sink_|.
+ } else if (agent_name == kArcTracingAgentName) {
+ // The ARC events are kept into a JSON format stored as string
+ // and must not be escaped.
+ trace_data_sink_->AddTraceChunk(events_str_ptr->data());
+ } else {
trace_data_sink_->AddAgentTrace(
events_label, base::GetQuotedJSONString(events_str_ptr->data()));
}
« content/browser/tracing/arc_tracing_agent.cc ('K') | « content/browser/tracing/arc_tracing_agent.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698