Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(420)

Unified Diff: content/content_tests.gypi

Issue 560553005: Battery impact UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix component build. Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/content_tests.gypi
diff --git a/content/content_tests.gypi b/content/content_tests.gypi
index 1073fc5f3fbc470377caa6dc0637356192d1668a..192704d773ca64ddd6b4f74aee3bee309ab2aa96 100644
--- a/content/content_tests.gypi
+++ b/content/content_tests.gypi
@@ -357,7 +357,11 @@
'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 y
jeremy 2014/11/17 07:50:48 No, this is the minimal set of dependencies needed
ppi 2014/11/17 10:51:01 Yup, it seems that we need the environment for log
'../net/net.gyp:net_test_support',
'../skia/skia.gyp:skia',
'../sql/sql.gyp:sql',
@@ -537,6 +541,7 @@
'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',

Powered by Google App Engine
This is Rietveld 408576698