Title: [179905] branches/safari-600.1.4.15-branch/Source/WebCore

Diff

Modified: branches/safari-600.1.4.15-branch/Source/WebCore/ChangeLog (179904 => 179905)


--- branches/safari-600.1.4.15-branch/Source/WebCore/ChangeLog	2015-02-11 00:54:20 UTC (rev 179904)
+++ branches/safari-600.1.4.15-branch/Source/WebCore/ChangeLog	2015-02-11 01:06:01 UTC (rev 179905)
@@ -1,5 +1,30 @@
 2015-02-10  Babak Shafiei  <bshaf...@apple.com>
 
+        Merge r179843.
+
+    2015-02-09  Jer Noble  <jer.no...@apple.com>
+
+            [Mac] Disable the currentTime estimation code in HTMLMediaElement for Yosemite+
+            https://bugs.webkit.org/show_bug.cgi?id=141399
+
+            Reviewed by Eric Carlson.
+
+            Apparenty -[AVPlayer rate] means different things for HLS and progressive content; for progressive,
+            the -rate is the actual rate of playback. For HLS, the -rate is the requested rate, and will return
+            the requested value even if time is not progressing.
+
+            We added the currentTime estimation engine because asking AVFoundation for its -currentTime used to
+            be expensive, but we've been assured that in recent iOS and OS X releases, -currentTime should be
+            very fast. That, in combination with the HLS behavior of -rate and how it breaks the currentTime
+            estimation, means we should probably turn it off for iOS and Yosemite.
+
+            * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h:
+            (WebCore::MediaPlayerPrivateAVFoundationObjC::maximumDurationToCacheMediaTime): Move implementation to .mm.
+            * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+            (WebCore::MediaPlayerPrivateAVFoundationObjC::maximumDurationToCacheMediaTime): Disable on iOS and >=10.10.
+
+2015-02-10  Babak Shafiei  <bshaf...@apple.com>
+
         Merge r178571.
 
     2015-01-15  Ryosuke Niwa  <rn...@webkit.org>

Modified: branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h (179904 => 179905)


--- branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h	2015-02-11 00:54:20 UTC (rev 179904)
+++ branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h	2015-02-11 01:06:01 UTC (rev 179905)
@@ -164,7 +164,7 @@
 
     virtual bool supportsAcceleratedRendering() const { return true; }
     virtual float mediaTimeForTimeValue(float) const;
-    virtual double maximumDurationToCacheMediaTime() const { return 5; }
+    virtual double maximumDurationToCacheMediaTime() const;
 
     virtual void createAVPlayer();
     virtual void createAVPlayerItem();

Modified: branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (179904 => 179905)


--- branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2015-02-11 00:54:20 UTC (rev 179904)
+++ branches/safari-600.1.4.15-branch/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm	2015-02-11 01:06:01 UTC (rev 179905)
@@ -1599,6 +1599,15 @@
     return timeValue;
 }
 
+double MediaPlayerPrivateAVFoundationObjC::maximumDurationToCacheMediaTime() const
+{
+#if PLATFORM(IOS) || __MAC_OS_X_VERSION_MIN_REQUIRED >= 1010
+    return 0;
+#else
+    return 5;
+#endif
+}
+
 void MediaPlayerPrivateAVFoundationObjC::updateVideoLayerGravity()
 {
     if (!m_videoLayer)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to