Makes the BattOrAgent asynchronous

tools/battor_agent/battor_agent.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "tools/battor_agent/battor_agent.h"
 
 #include <iostream>
 
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+
 namespace battor {
 
 BattOrAgent::BattOrAgent(const std::string& path) : path_(path) {
   // TODO: Open up a serial connection with the BattOr.
 }
 
 BattOrAgent::~BattOrAgent() {
   // TODO: Close the serial connection with the BattOr.
 }
 
 void BattOrAgent::StartTracing() {
   // TODO: Tell the BattOr to start tracing.
 }
 
-void BattOrAgent::StopTracing(std::string* out_trace) {
+void BattOrAgent::StopTracing(
+    const base::Callback<void(scoped_ptr<std::string>)>& callback) {
   // TODO: Tell the BattOr to stop tracing.
+  callback.Run(scoped_ptr<std::string>(new std::string("battor trace output")));

[Zhen Wang, 2015/10/29 19:40:22]

I think this callback run eventually should be moved to somewhere else, right?
For example, after BattOr sends trace data back?

[charliea (OOO until 10-5), 2015/10/29 21:12:11]

Yea. 

This callback will likely be sent in as the callback for SerialIOHandler::Write
(https://code.google.com/p/chromium/codesearch#chromium/src/device/serial/seri...).

 }
 
 void BattOrAgent::RecordClockSyncMarker(const std::string& marker) {
   // TODO: Tell the BattOr to record the specified clock sync marker.
 }
 
 void BattOrAgent::IssueClockSyncMarker() {
   // TODO: Tell atrace to issue a clock sync marker.
 }
 
 }  // namespace battor