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

Side by Side Diff: third_party/WebKit/LayoutTests/battery-status/restricted-level-precision.html

Issue 1844033004: Revert of Change the battery-status layout tests to use JS mocks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <html> 2 <html>
3 <body> 3 <body>
4 <script src="../resources/js-test.js"></script> 4 <script src="../resources/js-test.js"></script>
5 <script src="../resources/mojo-helpers.js"></script>
6 <script src="resources/mock-battery-monitor.js"></script>
7 <script> 5 <script>
8 description("Test to ensure level is reported with restricted precision."); 6 description("Test to ensure level is reported with restricted precision.");
9 7
10 if (!window.testRunner) 8 if (!window.testRunner)
11 debug('This test cannot be run without the TestRunner'); 9 debug('This test cannot be run without the TestRunner');
10 if (!window.internals)
11 debug('This test cannot be run without the window.internals');
12 12
13 // Clean-up any unused battery manager objects from previous tests. 13 // Clean-up any unused battery manager objects from previous tests.
14 gc(); 14 gc();
15 jsTestIsAsync = true; 15 jsTestIsAsync = true;
16 testRunner.waitUntilDone(); 16 testRunner.waitUntilDone();
17 17
18 var levelFullPrecision = 0.556789; 18 var levelFullPrecision = 0.556789;
19 var levelRounded = 0.56; 19 var levelRounded = 0.56;
20 20
21 function setAndFireMockBatteryInfo(charging, chargingTime, dischargingTime, leve l) {
22 window.internals.updateBatteryStatus(charging, chargingTime, dischargingTime , level);
23 }
24
21 var battery; 25 var battery;
22 function batteryStatusSuccess(batteryManager) { 26 function batteryStatusSuccess(batteryManager) {
23 debug('batteryStatusSuccess invoked'); 27 debug('batteryStatusSuccess invoked');
24 battery = batteryManager; 28 battery = batteryManager;
25 shouldBe('battery.level', 'levelRounded'); 29 shouldBe('battery.level', 'levelRounded');
26 setTimeout(finishJSTest, 0); 30 setTimeout(finishJSTest, 0);
27 } 31 }
28 32
29 mockBatteryMonitorReady.then(() => { 33 function batteryStatusFailure() {
30 navigator.getBattery().then(batteryStatusSuccess, batteryStatusFailure); 34 testFailed('failed to successfully resolve the promise');
31 setAndFireMockBatteryInfo(false, 10, 20, levelFullPrecision); 35 setTimeout(finishJSTest, 0);
32 }); 36 }
37
38 navigator.getBattery().then(batteryStatusSuccess, batteryStatusFailure);
39 setAndFireMockBatteryInfo(false, 10, 20, levelFullPrecision);
33 </script> 40 </script>
34 </body> 41 </body>
35 </html> 42 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698