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

Unified Diff: chrome/browser/chromeos/status/power_menu_button_browsertest.cc

Issue 6904160: Implement new gray mock. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed power test Created 9 years, 8 months 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: chrome/browser/chromeos/status/power_menu_button_browsertest.cc
diff --git a/chrome/browser/chromeos/status/power_menu_button_browsertest.cc b/chrome/browser/chromeos/status/power_menu_button_browsertest.cc
index 52ab1af06b6d8dbdad50207b5a20df0ccd8228b8..b39f82100bd814ef99b78be2dd61c48886d7ce26 100644
--- a/chrome/browser/chromeos/status/power_menu_button_browsertest.cc
+++ b/chrome/browser/chromeos/status/power_menu_button_browsertest.cc
@@ -93,7 +93,7 @@ IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryChargedTest) {
}
IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryChargingTest) {
- const int NUM_TIMES = 16;
+ const int NUM_TIMES = 19;
EXPECT_CALL(*mock_power_library_, battery_is_present())
.Times(NUM_TIMES)
.WillRepeatedly((Return(true)))
@@ -135,9 +135,12 @@ IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryChargingTest) {
IDR_STATUSBAR_BATTERY_CHARGING_14,
IDR_STATUSBAR_BATTERY_CHARGING_15,
IDR_STATUSBAR_BATTERY_CHARGING_16,
+ IDR_STATUSBAR_BATTERY_CHARGING_17,
+ IDR_STATUSBAR_BATTERY_CHARGING_18,
+ IDR_STATUSBAR_BATTERY_CHARGING_19,
};
size_t id = 0;
- for (float percent = 6.0; percent < 100.0; percent += 6.0) {
+ for (float percent = 5.0; percent < 100.0; percent += 5.0) {
EXPECT_CALL(*mock_power_library_, battery_percentage())
.WillOnce((Return(percent)))
.RetiresOnSaturation();
@@ -148,7 +151,7 @@ IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryChargingTest) {
}
IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryDischargingTest) {
- const int NUM_TIMES = 16;
+ const int NUM_TIMES = 19;
EXPECT_CALL(*mock_power_library_, battery_is_present())
.Times(NUM_TIMES)
.WillRepeatedly((Return(true)))
@@ -190,9 +193,12 @@ IN_PROC_BROWSER_TEST_F(PowerMenuButtonTest, BatteryDischargingTest) {
IDR_STATUSBAR_BATTERY_DISCHARGING_14,
IDR_STATUSBAR_BATTERY_DISCHARGING_15,
IDR_STATUSBAR_BATTERY_DISCHARGING_16,
+ IDR_STATUSBAR_BATTERY_DISCHARGING_17,
+ IDR_STATUSBAR_BATTERY_DISCHARGING_18,
+ IDR_STATUSBAR_BATTERY_DISCHARGING_19,
};
size_t id = 0;
- for (float percent = 6.0; percent < 100.0; percent += 6.0) {
+ for (float percent = 5.0; percent < 100.0; percent += 5.0) {
EXPECT_CALL(*mock_power_library_, battery_percentage())
.WillOnce((Return(percent)))
.RetiresOnSaturation();

Powered by Google App Engine
This is Rietveld 408576698