From 5e092ac44548ebc8bb85333d81ff29e8fc154c4a Mon Sep 17 00:00:00 2001
From: Paul Adenot <paul@paul.cx>
Date: Sat, 9 Jun 2012 17:07:19 -0400
Subject: [PATCH] Bug 762933 - No-audio icon switches to audio available icon
 when video is in fullscreen. r=jaws

---
 toolkit/content/widgets/videocontrols.xml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/toolkit/content/widgets/videocontrols.xml b/toolkit/content/widgets/videocontrols.xml
index 5a2c81e2af0..4aabfa732dd 100644
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -411,11 +411,19 @@
                     this.maxCurrentTimeSeen = currentTime;
                     this.showPosition(currentTime, duration);
 
-                    // If we have metadata, check if this is a <video> without video data.
+                    // If we have metadata, check if this is a <video> without
+                    // video data, or a video with no audio track.
                     if (this.video.readyState >= this.video.HAVE_METADATA) {
                         if (this.video instanceof HTMLVideoElement &&
                             (this.video.videoWidth == 0 || this.videoHeight == 0))
                             this.isAudioOnly = true;
+
+                        // We have to check again if the media has audio here,
+                        // because of bug 718107: switching to fullscreen may
+                        // cause the bindings to detach and reattach, hence
+                        // unsetting the attribute.
+                        if (!this.isAudioOnly && !this.video.mozHasAudio)
+                          this.muteButton.setAttribute("noAudio", "true");
                     }
 
                     if (this.isAudioOnly)