REGRESSION: AirPlay placard is not shown when in fullscreen
https://bugs.webkit.org/show_bug.cgi?id=173447
<rdar://problem/32803773>
Patch by Antoine Quint <graouts@apple.com> on 2017-06-16
Reviewed by Jon Lee.
Source/WebCore:
We only allowed a placard to be set on inline controls, but that was an oversight: placards
should be displayed in fullscreen as well. As such, we move the "placard" property up from
InlineMediaControls to MediaControls, and update the layout() logic in MacOSFullscreenMediaControls
to display a placard.
Test: media/modern-media-controls/placard-support/placard-support-airplay-fullscreen.html
* Modules/modern-media-controls/controls/inline-media-controls.js:
(InlineMediaControls):
(InlineMediaControls.prototype.layout):
(InlineMediaControls.prototype.get placard): Deleted.
(InlineMediaControls.prototype.set placard): Deleted.
* Modules/modern-media-controls/controls/macos-fullscreen-media-controls.js:
(MacOSFullscreenMediaControls.prototype.layout):
* Modules/modern-media-controls/controls/media-controls.js:
(MediaControls.prototype.get placard):
(MediaControls.prototype.set placard):
(MediaControls.prototype.placardPreventsControlsBarDisplay):
(MediaControls.prototype.layout):
* Modules/modern-media-controls/media/placard-support.js:
(PlacardSupport.prototype._updatePlacard):
(PlacardSupport):
LayoutTests:
Add a new test to check that we display the AirPlay placard on macOS in fullscreen.
* media/modern-media-controls/placard-support/placard-support-airplay-fullscreen-expected.txt: Added.
* media/modern-media-controls/placard-support/placard-support-airplay-fullscreen.html: Added.
* platform/ios-simulator/TestExpectations:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@218400 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index e07e181..e139678 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,5 +1,19 @@
2017-06-16 Antoine Quint <graouts@apple.com>
+ REGRESSION: AirPlay placard is not shown when in fullscreen
+ https://bugs.webkit.org/show_bug.cgi?id=173447
+ <rdar://problem/32803773>
+
+ Reviewed by Jon Lee.
+
+ Add a new test to check that we display the AirPlay placard on macOS in fullscreen.
+
+ * media/modern-media-controls/placard-support/placard-support-airplay-fullscreen-expected.txt: Added.
+ * media/modern-media-controls/placard-support/placard-support-airplay-fullscreen.html: Added.
+ * platform/ios-simulator/TestExpectations:
+
+2017-06-16 Antoine Quint <graouts@apple.com>
+
Backdrop blur missing in media controls bar on Sierra
https://bugs.webkit.org/show_bug.cgi?id=173451
