[ChromeDriver] Expose CPU Profiling DevTools API into chromeWebDriver

chrome/test/chromedriver/chrome/cpu_profile.h

 // Copyright 2013 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.
 
-#ifndef CHROME_TEST_CHROMEDRIVER_CHROME_HEAP_SNAPSHOT_TAKER_H_
-#define CHROME_TEST_CHROMEDRIVER_CHROME_HEAP_SNAPSHOT_TAKER_H_
+#ifndef CHROME_TEST_CHROMEDRIVER_CHROME_CPU_PROFILE_TAKER_H_
+#define CHROME_TEST_CHROMEDRIVER_CHROME_CPU_PROFILE_TAKER_H_
 
 #include <string>
-
-#include "base/basictypes.h"
+#include "base/logging.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/test/chromedriver/chrome/devtools_event_listener.h"
+#include "chrome/test/chromedriver/chrome/status.h"
 
 namespace base {
 class DictionaryValue;
 class Value;
 }
 
 class DevToolsClient;
 class Status;
 
 // Take the heap snapshot.
-class HeapSnapshotTaker: public DevToolsEventListener {
+class CpuProfile: public DevToolsEventListener {

[klm, 2014/05/10 02:21:57]

I believe the C++ style wants a space before the ":". I.e. HeapSnapshotTaker may
be a violation too.

  public:
-  explicit HeapSnapshotTaker(DevToolsClient* client);
-  virtual ~HeapSnapshotTaker();
+  explicit CpuProfile(DevToolsClient* client);
+  virtual ~CpuProfile();
 
-  Status TakeSnapshot(scoped_ptr<base::Value>* snapshot);
+  Status InitProfileInternal();

[klm, 2014/05/10 02:21:57]

These should be private.

+  Status StopProfileInternal();
+  Status StartProfile(scoped_ptr<base::Value>* snapshot);
+  Status EndProfile(scoped_ptr<base::Value>* snapshot);
 
   // Overridden from DevToolsEventListener:
   virtual Status OnEvent(DevToolsClient* client,
                          const std::string& method,
                          const base::DictionaryValue& params) OVERRIDE;
 
  private:
-  Status TakeSnapshotInternal();
+  Status CpuProfileInternal();

[klm, 2014/05/10 02:21:57]

This method seems to have no implementation defined.

 
   DevToolsClient* client_;
   std::string snapshot_;
 
-  DISALLOW_COPY_AND_ASSIGN(HeapSnapshotTaker);
+  DISALLOW_COPY_AND_ASSIGN(CpuProfile);
 };
 
-#endif  // CHROME_TEST_CHROMEDRIVER_CHROME_HEAP_SNAPSHOT_TAKER_H_
+#endif  // CHROME_TEST_CHROMEDRIVER_CHROME_CPU_PROFILE_TAKER_H_