From a325c5ee9c6551dd2b28d059e53ea18d180ee4a7 Mon Sep 17 00:00:00 2001
From: Justin Dolske <dolske@mozilla.com>
Date: Sat, 20 Dec 2008 14:57:34 -0800
Subject: [PATCH] Bug 469030 - video control binding shouldn't use ID
 attribute. r=enn

---
 toolkit/content/widgets/videocontrols.xml               | 14 +++++++-------
 toolkit/themes/pinstripe/global/media/videocontrols.css | 10 +++++-----
 toolkit/themes/winstripe/global/media/videocontrols.css | 10 +++++-----
 3 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/toolkit/content/widgets/videocontrols.xml b/toolkit/content/widgets/videocontrols.xml
index 304306562..67ca77c88 100644
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -14,11 +14,11 @@
 
     <xbl:content xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
         <spacer flex="1"/>
-        <hbox id="controlBar">
-            <image id="playButton"
+        <hbox class="controlBar">
+            <image class="playButton"
                    onclick="if (event.button == 0) this.parentNode.parentNode.Utils.togglePause();"/>
-            <box id="controlsMiddle" flex="1"/>
-            <image id="muteButton"
+            <box class="controlsMiddle" flex="1"/>
+            <image class="muteButton"
                    onclick="if (event.button == 0) this.parentNode.parentNode.Utils.toggleMute();"/>
         </hbox>
     </xbl:content>
@@ -226,9 +226,9 @@
             var video = this.parentNode;
             this.Utils.video = video;
 
-            this.Utils.controlBar = document.getAnonymousElementByAttribute(this, "id", "controlBar");
-            this.Utils.playButton = document.getAnonymousElementByAttribute(this, "id", "playButton");
-            this.Utils.muteButton = document.getAnonymousElementByAttribute(this, "id", "muteButton");
+            this.Utils.controlBar = document.getAnonymousElementByAttribute(this, "class", "controlBar");
+            this.Utils.playButton = document.getAnonymousElementByAttribute(this, "class", "playButton");
+            this.Utils.muteButton = document.getAnonymousElementByAttribute(this, "class", "muteButton");
 
             // Set initial state of play/pause button.
             this.Utils.playButton.setAttribute("paused", video.paused);
diff --git a/toolkit/themes/pinstripe/global/media/videocontrols.css b/toolkit/themes/pinstripe/global/media/videocontrols.css
index 70f2bab50..bd1755c5c 100644
--- a/toolkit/themes/pinstripe/global/media/videocontrols.css
+++ b/toolkit/themes/pinstripe/global/media/videocontrols.css
@@ -1,23 +1,23 @@
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
-#controlBar {
+.controlBar {
     height: 24px;
     background-color: #656363;
 }
 
-#playButton {
+.playButton {
     list-style-image: url(chrome://global/skin/media/videocontrols.png);
     -moz-image-region: rect(0px, 48px, 24px, 24px);
 }
-#playButton[paused="true"] {
+.playButton[paused="true"] {
     -moz-image-region: rect(0px, 24px, 24px, 0px);
 }
 
-#muteButton{
+.muteButton {
     list-style-image: url(chrome://global/skin/media/videocontrols.png);
     -moz-image-region: rect(24px, 24px, 48px, 0px);
 }
-#muteButton[muted="true"] {
+.muteButton[muted="true"] {
     -moz-image-region: rect(24px, 48px, 48px, 24px);
 }
 
diff --git a/toolkit/themes/winstripe/global/media/videocontrols.css b/toolkit/themes/winstripe/global/media/videocontrols.css
index 70f2bab50..bd1755c5c 100644
--- a/toolkit/themes/winstripe/global/media/videocontrols.css
+++ b/toolkit/themes/winstripe/global/media/videocontrols.css
@@ -1,23 +1,23 @@
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
-#controlBar {
+.controlBar {
     height: 24px;
     background-color: #656363;
 }
 
-#playButton {
+.playButton {
     list-style-image: url(chrome://global/skin/media/videocontrols.png);
     -moz-image-region: rect(0px, 48px, 24px, 24px);
 }
-#playButton[paused="true"] {
+.playButton[paused="true"] {
     -moz-image-region: rect(0px, 24px, 24px, 0px);
 }
 
-#muteButton{
+.muteButton {
     list-style-image: url(chrome://global/skin/media/videocontrols.png);
     -moz-image-region: rect(24px, 24px, 48px, 0px);
 }
-#muteButton[muted="true"] {
+.muteButton[muted="true"] {
     -moz-image-region: rect(24px, 48px, 48px, 24px);
 }
 
-- 
2.11.4.GIT