[ChromeDriver] Subscribe PerformanceLogger to CommandListener interface

chrome/chrome_tests.gypi

 # 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.
 {
   'includes': [
     'js_unittest_vars.gypi',
   ],
   'targets': [
     {
       'target_name': 'interactive_ui_tests',
@@ skipping 440 matching lines @@
         'test/chromedriver/chrome/javascript_dialog_manager.cc',
         'test/chromedriver/chrome/javascript_dialog_manager.h',
         'test/chromedriver/chrome/log.h',
         'test/chromedriver/chrome/log.cc',
         'test/chromedriver/chrome/mobile_device.cc',
         'test/chromedriver/chrome/mobile_device.h',
         'test/chromedriver/chrome/mobile_emulation_override_manager.cc',
         'test/chromedriver/chrome/mobile_emulation_override_manager.h',
         'test/chromedriver/chrome/navigation_tracker.cc',
         'test/chromedriver/chrome/navigation_tracker.h',
-        'test/chromedriver/chrome/performance_logger.h',
-        'test/chromedriver/chrome/performance_logger.cc',
         'test/chromedriver/chrome/status.cc',
         'test/chromedriver/chrome/status.h',
         'test/chromedriver/chrome/ui_events.cc',
         'test/chromedriver/chrome/ui_events.h',
         'test/chromedriver/chrome/util.cc',
         'test/chromedriver/chrome/util.h',
         'test/chromedriver/chrome/version.cc',
         'test/chromedriver/chrome/version.h',
         'test/chromedriver/chrome/web_view.h',
         'test/chromedriver/chrome/web_view_impl.cc',
@@ skipping 139 matching lines @@
         'common/chrome_constants.cc',
         'common/chrome_constants.h',
         'test/chromedriver/alert_commands.cc',
         'test/chromedriver/alert_commands.h',
         'test/chromedriver/basic_types.cc',
         'test/chromedriver/basic_types.h',
         'test/chromedriver/capabilities.cc',
         'test/chromedriver/capabilities.h',
         'test/chromedriver/chrome_launcher.cc',
         'test/chromedriver/chrome_launcher.h',
+        'test/chromedriver/command_listener_proxy.cc',
+        'test/chromedriver/command_listener_proxy.h',
+        'test/chromedriver/command_listener.h',

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

Seems like command_listener.h should go first as web_view.h

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

Done.

         'test/chromedriver/command.h',
         'test/chromedriver/commands.cc',
         'test/chromedriver/commands.h',
         'test/chromedriver/element_commands.cc',
         'test/chromedriver/element_commands.h',
         'test/chromedriver/element_util.cc',
         'test/chromedriver/element_util.h',
         'test/chromedriver/key_converter.cc',
         'test/chromedriver/key_converter.h',
         'test/chromedriver/keycode_text_conversion.h',
         'test/chromedriver/keycode_text_conversion_mac.mm',
         'test/chromedriver/keycode_text_conversion_ozone.cc',
         'test/chromedriver/keycode_text_conversion_win.cc',
         'test/chromedriver/keycode_text_conversion_x.cc',
         'test/chromedriver/logging.cc',
         'test/chromedriver/logging.h',
+        'test/chromedriver/performance_logger.h',
+        'test/chromedriver/performance_logger.cc',

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

It seems the original order was wrong.
Would you mind correcting it here?

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

Done.

         'test/chromedriver/server/http_handler.cc',
         'test/chromedriver/server/http_handler.h',
         'test/chromedriver/session.cc',
         'test/chromedriver/session.h',
         'test/chromedriver/session_commands.cc',
         'test/chromedriver/session_commands.h',
         'test/chromedriver/session_thread_map.h',
         'test/chromedriver/util.cc',
         'test/chromedriver/util.h',
         'test/chromedriver/window_commands.cc',
@@ skipping 76 matching lines @@
         'test/chromedriver/chrome/device_manager_unittest.cc',
         'test/chromedriver/chrome/devtools_client_impl_unittest.cc',
         'test/chromedriver/chrome/devtools_http_client_unittest.cc',
         'test/chromedriver/chrome/dom_tracker_unittest.cc',
         'test/chromedriver/chrome/frame_tracker_unittest.cc',
         'test/chromedriver/chrome/geolocation_override_manager_unittest.cc',
         'test/chromedriver/chrome/heap_snapshot_taker_unittest.cc',
         'test/chromedriver/chrome/javascript_dialog_manager_unittest.cc',
         'test/chromedriver/chrome/mobile_emulation_override_manager_unittest.cc',
         'test/chromedriver/chrome/navigation_tracker_unittest.cc',
-        'test/chromedriver/chrome/performance_logger_unittest.cc',
         'test/chromedriver/chrome/status_unittest.cc',
         'test/chromedriver/chrome/stub_chrome.cc',
         'test/chromedriver/chrome/stub_chrome.h',
         'test/chromedriver/chrome/stub_devtools_client.cc',
         'test/chromedriver/chrome/stub_devtools_client.h',
         'test/chromedriver/chrome/stub_web_view.cc',
         'test/chromedriver/chrome/stub_web_view.h',
         'test/chromedriver/chrome/web_view_impl_unittest.cc',
         'test/chromedriver/chrome_launcher_unittest.cc',
+        'test/chromedriver/command_listener_proxy_unittest.cc',
         'test/chromedriver/commands_unittest.cc',
         'test/chromedriver/logging_unittest.cc',
+        'test/chromedriver/performance_logger_unittest.cc',
         'test/chromedriver/server/http_handler_unittest.cc',
         'test/chromedriver/session_commands_unittest.cc',
         'test/chromedriver/session_unittest.cc',
         'test/chromedriver/util_unittest.cc',
       ],
       # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
       'msvs_disabled_warnings': [ 4267, ],
     },
     # ChromeDriver tests that aren't run on the main buildbot. Available
     # as an optional test type on trybots.
@@ skipping 2208 matching lines @@
             ['enable_webrtc==1', {
               'dependencies': [
                 '../third_party/libjingle/libjingle.gyp:libjingle_webrtc'
               ]
             }],
           ],
         }]
     }],
   ],  # 'conditions'
 }