Title: [220910] trunk
- Revision
- 220910
- Author
- [email protected]
- Date
- 2017-08-18 01:58:34 -0700 (Fri, 18 Aug 2017)
Log Message
[GTK] Show controls if a video element isn't allowed to play inline.
https://bugs.webkit.org/show_bug.cgi?id=141705
Patch by Ms2ger <[email protected]> on 2017-08-18
Reviewed by Xabier Rodriguez-Calvar.
Source/WebCore:
Test: covered by media/video-fullscreeen-only-controls.html [sic].
* Modules/mediacontrols/mediaControlsGtk.js:
(ControllerGtk.prototype.shouldHaveControls):
LayoutTests:
* platform/gtk/TestExpectations: enable test for this case.
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (220909 => 220910)
--- trunk/LayoutTests/ChangeLog 2017-08-18 07:53:06 UTC (rev 220909)
+++ trunk/LayoutTests/ChangeLog 2017-08-18 08:58:34 UTC (rev 220910)
@@ -1,3 +1,12 @@
+2017-08-18 Ms2ger <[email protected]>
+
+ [GTK] Show controls if a video element isn't allowed to play inline.
+ https://bugs.webkit.org/show_bug.cgi?id=141705
+
+ Reviewed by Xabier Rodriguez-Calvar.
+
+ * platform/gtk/TestExpectations: enable test for this case.
+
2017-08-17 Wenson Hsieh <[email protected]>
[Input Events] Rename inputType "formatForeColor" to "formatFontColor"
Modified: trunk/LayoutTests/platform/gtk/TestExpectations (220909 => 220910)
--- trunk/LayoutTests/platform/gtk/TestExpectations 2017-08-18 07:53:06 UTC (rev 220909)
+++ trunk/LayoutTests/platform/gtk/TestExpectations 2017-08-18 08:58:34 UTC (rev 220910)
@@ -919,8 +919,6 @@
webkit.org/b/141699 fast/attachment/attachment-rendering.html [ Failure ]
webkit.org/b/141699 fast/attachment/attachment-respects-css-size.html [ ImageOnlyFailure ]
-webkit.org/b/141705 media/video-fullscreeen-only-controls.html [ Failure ]
-
webkit.org/b/141831 svg/animations/animate-text-nested-transforms.html [ Failure ]
webkit.org/b/98653 svg/text/lengthAdjust-text-metrics.html [ Failure ]
Modified: trunk/Source/WebCore/ChangeLog (220909 => 220910)
--- trunk/Source/WebCore/ChangeLog 2017-08-18 07:53:06 UTC (rev 220909)
+++ trunk/Source/WebCore/ChangeLog 2017-08-18 08:58:34 UTC (rev 220910)
@@ -1,3 +1,15 @@
+2017-08-18 Ms2ger <[email protected]>
+
+ [GTK] Show controls if a video element isn't allowed to play inline.
+ https://bugs.webkit.org/show_bug.cgi?id=141705
+
+ Reviewed by Xabier Rodriguez-Calvar.
+
+ Test: covered by media/video-fullscreeen-only-controls.html [sic].
+
+ * Modules/mediacontrols/mediaControlsGtk.js:
+ (ControllerGtk.prototype.shouldHaveControls):
+
2017-08-18 Zan Dobersek <[email protected]>
[EME] MediaKeySession has to initiate playback resume on HTMLMediaElement
Modified: trunk/Source/WebCore/Modules/mediacontrols/mediaControlsGtk.js (220909 => 220910)
--- trunk/Source/WebCore/Modules/mediacontrols/mediaControlsGtk.js 2017-08-18 07:53:06 UTC (rev 220909)
+++ trunk/Source/WebCore/Modules/mediacontrols/mediaControlsGtk.js 2017-08-18 08:58:34 UTC (rev 220910)
@@ -56,6 +56,9 @@
shouldHaveControls: function()
{
+ if (!this.isAudio() && !this.host.allowsInlineMediaPlayback)
+ return true;
+
return this.video.controls || this.isFullScreen();
},
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
