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

Reply via email to