Battery impact UMA

content/content_tests.gypi

 # Copyright (c) 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.
 
 {
   'variables': {
     'layouttest_support_content_sources': [
       'public/test/layouttest_support.h',
       'public/test/nested_message_pump_android.cc',
       'public/test/nested_message_pump_android.h',
@@ skipping 339 matching lines @@
       'target_name': 'content_unittests',
       'type': '<(gtest_target_type)',
       'dependencies': [
         'browser/service_worker/service_worker_proto.gyp:proto',
         'browser/speech/proto/speech_proto.gyp:speech_proto',
         'content.gyp:content_browser',
         'content.gyp:content_common',
         'test_support_content',
         '../base/base.gyp:test_support_base',
         '../crypto/crypto.gyp:crypto',
+        '../device/battery/battery.gyp:device_battery',
+        '../device/battery/battery.gyp:device_battery_mojo_bindings',
         '../mojo/edk/mojo_edk.gyp:mojo_common_test_support',
+        '../mojo/mojo_base.gyp:mojo_environment_chromium',
+        '../mojo/public/mojo_public.gyp:mojo_cpp_bindings',

[ppi, 2014/11/16 16:03:08]

Does it compile without these two? It seems that you're using only the generated
datatype in the tests, hence the ones with "battery.gyp" might be enough.

[jeremy, 2014/11/17 07:50:48]

No, this is the minimal set of dependencies needed for this to compile, I get
the following error when removing these:

Undefined symbols for architecture i386:
  "__ZN4mojo8internal10LogMessageC1EPKcii", referenced from:
      __ZNK4mojo16InlinedStructPtrIN6device13BatteryStatusEEptEv in
libdevice_battery_mojo_bindings.a(device_battery_mojo_bindings.battery_status.mojom.o)
  "__ZN4mojo8internal10LogMessageD1Ev", referenced from:
      __ZNK4mojo16InlinedStructPtrIN6device13BatteryStatusEEptEv in
libdevice_battery_mojo_bindings.a(device_battery_mojo_bindings.battery_status.mojom.o)
  "__ZN4mojo8internal20ValidateStructHeaderEPKvjjPNS0_13BoundsCheckerE",
referenced from:
     
__ZN6device8internal18BatteryStatus_Data8ValidateEPKvPN4mojo8internal13BoundsCheckerE
in
libdevice_battery_mojo_bindings.a(device_battery_mojo_bindings.battery_status.mojom.o)

[ppi, 2014/11/17 10:51:01]

Yup, it seems that we need the environment for logging and cpp_bindings for
validation, both of which due to the generated battery_status.mojom h/cc code
that we're referencing in the test. Please go ahead.

         '../net/net.gyp:net_test_support',
         '../skia/skia.gyp:skia',
         '../sql/sql.gyp:sql',
         '../sql/sql.gyp:test_support_sql',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
         '../third_party/re2/re2.gyp:re2',
         '../ui/accessibility/accessibility.gyp:accessibility',
         '../ui/base/ui_base.gyp:ui_base',
         '../ui/gfx/gfx.gyp:gfx',
@@ skipping 159 matching lines @@
         'browser/media/capture/video_capture_oracle_unittest.cc',
         'browser/media/capture/web_contents_audio_input_stream_unittest.cc',
         'browser/media/capture/web_contents_video_capture_device_unittest.cc',
         'browser/media/media_internals_unittest.cc',
         'browser/media/midi_host_unittest.cc',
         'browser/media/webrtc_identity_store_unittest.cc',
         'browser/net/sqlite_persistent_cookie_store_unittest.cc',
         'browser/notification_service_impl_unittest.cc',
         'browser/plugin_loader_posix_unittest.cc',
         'browser/power_monitor_message_broadcaster_unittest.cc',
+        'browser/power_usage_monitor_impl_unittest.cc',
         'browser/power_profiler/power_profiler_service_unittest.cc',
         'browser/quota/mock_quota_manager.cc',
         'browser/quota/mock_quota_manager.h',
         'browser/quota/mock_quota_manager_proxy.cc',
         'browser/quota/mock_quota_manager_proxy.h',
         'browser/quota/mock_quota_manager_unittest.cc',
         'browser/quota/quota_database_unittest.cc',
         'browser/quota/quota_backend_impl_unittest.cc',
         'browser/quota/quota_manager_unittest.cc',
         'browser/quota/quota_reservation_manager_unittest.cc',
@@ skipping 1457 matching lines @@
                   'files': ['<(PRODUCT_DIR)/libnpapi_test_plugin.so'],
                 },
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }