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

Unified Diff: chrome/test/data/webui/media_router/route_controls_tests.js

Issue 2932933002: [Media Router] Increment the media's current_time in the WebUI route controller (Closed)
Patch Set: Check that we can increment in onRouteStatusChange_() Created 3 years, 6 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
« no previous file with comments | « chrome/common/media_router/mojo/media_status_struct_traits.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/media_router/route_controls_tests.js
diff --git a/chrome/test/data/webui/media_router/route_controls_tests.js b/chrome/test/data/webui/media_router/route_controls_tests.js
index 910c7a95b99506257f7dc70ca58f479893ad70ef..ccfebb4687216fea003206de54781ec263e4a8c8 100644
--- a/chrome/test/data/webui/media_router/route_controls_tests.js
+++ b/chrome/test/data/webui/media_router/route_controls_tests.js
@@ -47,6 +47,8 @@ cr.define('route_controls', function() {
params.title ? params.title : '',
params.status ? params.status : '', !!params.canPlayPause,
!!params.canMute, !!params.canSetVolume, !!params.canSeek,
+ params.playState ? params.playState :
+ media_router.PlayState.PLAYING,
!!params.isPaused, !!params.isMuted,
params.volume ? params.volume : 0,
params.duration ? params.duration : 0,
@@ -138,8 +140,8 @@ cr.define('route_controls', function() {
done();
});
- controls.routeStatus =
- createRouteStatus({canPlayPause: true, isPaused: true});
+ controls.routeStatus = createRouteStatus(
+ {canPlayPause: true, playState: media_router.PlayState.PAUSED});
MockInteractions.tap(controls.$$('#route-play-pause-button'));
});
@@ -149,8 +151,8 @@ cr.define('route_controls', function() {
done();
});
- controls.routeStatus =
- createRouteStatus({canPlayPause: true, isPaused: false});
+ controls.routeStatus = createRouteStatus(
+ {canPlayPause: true, playState: media_router.PlayState.PLAYING});
MockInteractions.tap(controls.$$('#route-play-pause-button'));
});
@@ -238,6 +240,30 @@ cr.define('route_controls', function() {
controls.$$('#route-volume-slider').value = volume;
controls.$$('#route-volume-slider').fire('change');
});
+
+ test('increment current time while playing', function(done) {
+ var initialTime = 50;
+ controls.routeStatus = createRouteStatus({
+ canSeek: true,
+ playState: media_router.PlayState.PLAYING,
+ duration: 100,
+ currentTime: initialTime,
+ });
+
+ // Check that the current time has been incremented after a second.
+ setTimeout(function() {
+ controls.routeStatus.playState = media_router.PlayState.PAUSED;
+ var pausedTime = controls.routeStatus.currentTime;
+ assertTrue(pausedTime > initialTime);
+
+ // Check that the current time stayed the same after a second, now
+ // that the media is paused.
+ setTimeout(function() {
+ assertEquals(pausedTime, controls.routeStatus.currentTime);
+ done();
+ }, 1000);
+ }, 1000);
+ });
});
}
« no previous file with comments | « chrome/common/media_router/mojo/media_status_struct_traits.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698