Title: [184963] trunk/Source/WebCore
Revision
184963
Author
eric.carl...@apple.com
Date
2015-05-28 15:31:17 -0700 (Thu, 28 May 2015)

Log Message

[iOS] Fix controls bug caused by r184794
https://bugs.webkit.org/show_bug.cgi?id=145445

Reviewed by Dean Jackson.

* Modules/mediacontrols/mediaControlsiOS.js:
(ControllerIOS.prototype.showControls): Check video.controls instead of trying to call
showInlinePlaybackPlaceholderOnly, which doesn't exist.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (184962 => 184963)


--- trunk/Source/WebCore/ChangeLog	2015-05-28 21:55:32 UTC (rev 184962)
+++ trunk/Source/WebCore/ChangeLog	2015-05-28 22:31:17 UTC (rev 184963)
@@ -1,3 +1,14 @@
+2015-05-28  Eric Carlson  <eric.carl...@apple.com>
+
+        [iOS] Fix controls bug caused by r184794
+        https://bugs.webkit.org/show_bug.cgi?id=145445
+
+        Reviewed by Dean Jackson.
+
+        * Modules/mediacontrols/mediaControlsiOS.js:
+        (ControllerIOS.prototype.showControls): Check video.controls instead of trying to call
+        showInlinePlaybackPlaceholderOnly, which doesn't exist.
+
 2015-05-28  Said Abou-Hallawa  <sabouhall...@apple.com>
 
         Code clean up for extracting information from the mix of WritingMode and TextDirection

Modified: trunk/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js (184962 => 184963)


--- trunk/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js	2015-05-28 21:55:32 UTC (rev 184962)
+++ trunk/Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js	2015-05-28 22:31:17 UTC (rev 184963)
@@ -541,7 +541,7 @@
     showControls: function()
     {
         this.updateShouldListenForPlaybackTargetAvailabilityEvent();
-        if (this.showInlinePlaybackPlaceholderOnly())
+        if (!this.video.controls)
             return;
         
         this.updateForShowingControls();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to