Issue 219523004: Let MediaController.currentTime return the position previously set

Issue 219523004: Let MediaController.currentTime return the position previously set (Closed)

Created:
6 years, 8 months ago by philipj_slow

Modified:
6 years, 8 months ago

Reviewers:
acolwell GONE FROM CHROMIUM

CC:
blink-reviews, feature-media-reviews_chromium.org, dglazkov+blink, philipj_slow, adamk+blink_chromium.org, eric.carlson_apple.com

Base URL:
svn://svn.chromium.org/blink/trunk

Visibility:
Public.

Let MediaController.currentTime return the position previously set BUG=358226 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170504

Created: 6 years, 8 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+3 lines, -0 lines)			Patch
M	LayoutTests/media/media-controller-playback.html	View	1 chunk	+1 line, -0 lines	0 comments	Download
M	LayoutTests/media/media-controller-playback-expected.txt	View	1 chunk	+1 line, -0 lines	0 comments	Download
M	Source/core/html/MediaController.cpp	View	1 chunk	+1 line, -0 lines	0 comments	Download

Total messages: 7 (0 generated)

philipj_slow

PTAL. This is required for the test in https://codereview.chromium.org/219283004/ to pass.

6 years, 8 months ago (2014-03-31 17:29:42 UTC) #1

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/219523004/1

6 years, 8 months ago (2014-04-01 01:33:56 UTC) #4

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/219523004/1

6 years, 8 months ago (2014-04-01 01:34:16 UTC) #5

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/219523004/1

6 years, 8 months ago (2014-04-01 01:34:36 UTC) #6

Message was sent while issue was closed.

Change committed as 170504