Title: [210424] branches/safari-602-branch/Source/WebCore
Revision
210424
Author
[email protected]
Date
2017-01-05 18:05:22 -0800 (Thu, 05 Jan 2017)

Log Message

Roll out r210328. rdar://problem/29885151

Modified Paths

Diff

Modified: branches/safari-602-branch/Source/WebCore/ChangeLog (210423 => 210424)


--- branches/safari-602-branch/Source/WebCore/ChangeLog	2017-01-06 01:55:07 UTC (rev 210423)
+++ branches/safari-602-branch/Source/WebCore/ChangeLog	2017-01-06 02:05:22 UTC (rev 210424)
@@ -1,5 +1,9 @@
 2017-01-05  Babak Shafiei  <[email protected]>
 
+        Roll out r210328.
+
+2017-01-05  Babak Shafiei  <[email protected]>
+
         Merge r210372.
 
     2017-01-05  Chris Dumez  <[email protected]>

Modified: branches/safari-602-branch/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm (210423 => 210424)


--- branches/safari-602-branch/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm	2017-01-06 01:55:07 UTC (rev 210423)
+++ branches/safari-602-branch/Source/WebCore/platform/mac/WebPlaybackSessionInterfaceMac.mm	2017-01-06 02:05:22 UTC (rev 210424)
@@ -188,10 +188,7 @@
         || (manager.rate < 0 && model->playbackStartedTime() <= currentTime))
         effectivePlaybackRate = 0;
 
-    // FIXME: The timing value should ideally use the effective playback rate, but this causes AVKit to indefinitely fire a repeating timer.
-    // More investigation will be required before we re-enable smooth playhead animation -- for now, we should just pretend that the playback
-    // rate is always 0.
-    manager.timing = [getAVValueTimingClass() valueTimingWithAnchorValue:currentTime anchorTimeStamp:effectiveAnchorTime rate:0];
+    manager.timing = [getAVValueTimingClass() valueTimingWithAnchorValue:currentTime anchorTimeStamp:effectiveAnchorTime rate:effectivePlaybackRate];
 }
 
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to