250594 2023-01-13 14:12:49 -0800 Let MediaController.currentTime return the position previously set 2023-01-18 01:24:16 -0800 1 1 1 Unclassified WebKit Media WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 ahmad.saleem792 webkit-unassigned ap bfulgham eric.carlson jer.noble webkit-bug-importer oldest_to_newest 1925445 0 ahmad.saleem792 2023-01-13 14:12:49 -0800 Hi Team, While going through Blink's commit, came across a bug, which is no align with web-spec: Blink Commit - https://chromium.googlesource.com/chromium/blink/+/32276c29774c6af8a4ad9f6d0d6551bf8323883c WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/html/MediaController.cpp#169 ''' The currentTime attribute must, on getting, return the media element's default playback start position, unless that is zero, in which case it must return the element's official playback position. The returned value must be expressed in seconds. On setting, if the media element's readyState is HAVE_NOTHING, then it must set the media element's default playback start position to the new value; otherwise, it must set the official playback position to the new value and then seek to the new value. The new value must be interpreted as being in seconds. ''' Web-Spec - https://html.spec.whatwg.org/multipage/media.html#offsets-into-the-media-resource Just wanted to raise to get input, whether we should go ahead and merge this. Thanks! 1925975 1 eric.carlson 2023-01-17 06:14:49 -0800 I think we should. 1925976 2 ahmad.saleem792 2023-01-17 06:16:25 -0800 (In reply to Eric Carlson from comment #1) > I think we should. Cool! I will do PR later today and see what happens with EWS. :-) 1926222 3 ews-feeder 2023-01-18 01:23:11 -0800 Committed 259020@main (921f8243c150): <https://commits.webkit.org/259020@main> Reviewed commits have been landed. Closing PR #8740 and removing active labels. 1926223 4 webkit-bug-importer 2023-01-18 01:24:16 -0800 <rdar://problem/104370834>