Title: [171934] tags/Safari-600.1.4.3/Source/WebCore
- Revision
- 171934
- Author
- matthew_han...@apple.com
- Date
- 2014-08-01 11:33:31 -0700 (Fri, 01 Aug 2014)
Log Message
Merge r171259. <rdar://problem/17718984>
Modified Paths
Diff
Modified: tags/Safari-600.1.4.3/Source/WebCore/ChangeLog (171933 => 171934)
--- tags/Safari-600.1.4.3/Source/WebCore/ChangeLog 2014-08-01 18:32:18 UTC (rev 171933)
+++ tags/Safari-600.1.4.3/Source/WebCore/ChangeLog 2014-08-01 18:33:31 UTC (rev 171934)
@@ -2,6 +2,21 @@
Merge r171259. <rdar://problem/17718984>
+ 2014-07-20 Eric Carlson <eric.carl...@apple.com>
+
+ [iOS] ignore requests to set volume
+ https://bugs.webkit.org/show_bug.cgi?id=135081
+
+ Applied post-review comments from Darin Adler.
+
+ * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+ (WebCore::MediaPlayerPrivateAVFoundationObjC::setVolume): Don't include unreachable
+ code on iOS.
+
+2014-07-31 Matthew Hanson <matthew_han...@apple.com>
+
+ Merge r171259. <rdar://problem/17718984>
+
2014-07-18 Eric Carlson <eric.carl...@apple.com>
[iOS] ignore requests to set volume
Modified: tags/Safari-600.1.4.3/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (171933 => 171934)
--- tags/Safari-600.1.4.3/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2014-08-01 18:32:18 UTC (rev 171933)
+++ tags/Safari-600.1.4.3/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2014-08-01 18:33:31 UTC (rev 171934)
@@ -1156,13 +1156,14 @@
void MediaPlayerPrivateAVFoundationObjC::setVolume(float volume)
{
#if PLATFORM(IOS)
+ UNUSED_PARAM(volume);
return;
-#endif
-
+#else
if (!metaDataAvailable())
return;
[m_avPlayer.get() setVolume:volume];
+#endif
}
void MediaPlayerPrivateAVFoundationObjC::setClosedCaptionsVisible(bool closedCaptionsVisible)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes