[ChromeDriver] Subscribe PerformanceLogger to CommandListener interface

chrome/test/chromedriver/command_listener_proxy.h

Index: chrome/test/chromedriver/command_listener_proxy.h
diff --git a/chrome/test/chromedriver/command_listener_proxy.h b/chrome/test/chromedriver/command_listener_proxy.h
new file mode 100644
index 0000000000000000000000000000000000000000..38fd3000187aaf1bc80bc681c8bf9d274eaee0ba
--- /dev/null
+++ b/chrome/test/chromedriver/command_listener_proxy.h
@@ -0,0 +1,30 @@
+// Copyright 2014 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_COMMAND_LISTENER_PROXY_H_
+#define CHROME_TEST_CHROMEDRIVER_COMMAND_LISTENER_PROXY_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "chrome/test/chromedriver/command_listener.h"
+
+class CommandListenerProxy : public CommandListener {
+ public:
+  virtual ~CommandListenerProxy();
+
+  // command_listener must not be null.

[stgao, 2014/07/01 19:02:26]

command_listener  ->  |command_listener|

[johnmoore, 2014/07/02 22:17:44]

Done.

+  explicit CommandListenerProxy(CommandListener* command_listener);
+
+  // Forwards commands to command_listener.

[stgao, 2014/07/01 19:02:27]

command_listener seems unclear to me: command_listener from constructor or
command_listener_?

Maybe just say "CommandListener"?

[johnmoore, 2014/07/02 22:17:44]

Done.

+  virtual Status BeforeCommand(const std::string& command_name);

[stgao, 2014/07/01 19:02:27]

OVERRIDE

[johnmoore, 2014/07/02 22:17:44]

Done.

+
+ private:
+  CommandListener* const command_listener_;
+
+  DISALLOW_COPY_AND_ASSIGN(CommandListenerProxy);
+};
+
+#endif  // CHROME_TEST_CHROMEDRIVER_COMMAND_LISTENER_PROXY_H_