[ChromeDriver] Subscribe PerformanceLogger to CommandListener interface

chrome/test/chromedriver/session.h

Index: chrome/test/chromedriver/session.h
diff --git a/chrome/test/chromedriver/session.h b/chrome/test/chromedriver/session.h
index fe837fbf061d252393742ef161326ad3cf8051fe..2ddfb81ad5b3cd0f8dfc4082363516f64ab36a42 100644
--- a/chrome/test/chromedriver/session.h
+++ b/chrome/test/chromedriver/session.h
@@ -16,6 +16,7 @@
 #include "chrome/test/chromedriver/basic_types.h"
 #include "chrome/test/chromedriver/chrome/device_metrics.h"
 #include "chrome/test/chromedriver/chrome/geoposition.h"
+#include "chrome/test/chromedriver/command_listener.h"
 
 namespace base {
 class DictionaryValue;
@@ -49,6 +50,7 @@ struct Session {
   void SwitchToParentFrame();
   void SwitchToSubFrame(const std::string& frame_id,
                         const std::string& chromedriver_frame_id);
+  void AddListener(CommandListener* listener);

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

Seems unused now.
Could be removed.

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

Done.

   std::string GetCurrentFrameId() const;
   std::vector<WebDriverLog*> GetAllLogs() const;
   std::string GetFirstBrowserError() const;
@@ -77,6 +79,11 @@ struct Session {
   base::ScopedTempDir temp_dir;
   scoped_ptr<base::DictionaryValue> capabilities;
   bool auto_reporting_enabled;
+  // |command_listeners| should be declared after |chrome|. When the |Session|
+  // is destroyed, |command_listeners| should be freed first, since some
+  // |CommandListener|s might be |CommandListenerProxy|s that forward to
+  // |DevToolsEventListener|s owned by |chrome|.
+  ScopedVector<CommandListener> command_listeners;
 };
 
 Session* GetThreadLocalSession();