Bug 1274146 part 1 - Not call setFullscreenButtonState() in fullscreenchange handler. r?gijs
Fullscreen change triggers resize reflow, and setFullscreenButtonState
should be called by setupInitialState() then when the video control is
rebound to the element.
MozReview-Commit-ID: 9B2IoNEPDeX
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -1092,17 +1092,16 @@
else
this.fullscreenButton.removeAttribute("fullscreened");
},
onFullscreenChange: function () {
if (this.isVideoInFullScreen()) {
Utils._hideControlsTimeout = setTimeout(this._hideControlsFn, this.HIDE_CONTROLS_TIMEOUT_MS);
}
- this.setFullscreenButtonState();
},
clickToPlayClickHandler : function(e) {
if (e.button != 0)
return;
if (this.hasError() && !this.suppressError) {
// Errors that can be dismissed should be placed here as we discover them.
if (this.video.error.code != this.video.error.MEDIA_ERR_ABORTED)