From dee9c96590f5821c5aeb7af7b18eb7cdd715bf8a Mon Sep 17 00:00:00 2001
From: Justin Dolske <dolske@mozilla.com>
Date: Sun, 12 Apr 2009 23:09:25 -0700
Subject: [PATCH] Bug 475317 - Need UI for volume control on <video>. r=enn,
 ui-r=boriss

---
 toolkit/content/widgets/videocontrols.css     |   2 +-
 toolkit/content/widgets/videocontrols.xml     | 107 +++++++++++++++---
 toolkit/themes/pinstripe/global/jar.mn        |   1 +
 .../pinstripe/global/media/videocontrols.css  |  36 +++++-
 .../pinstripe/global/media/volumeThumb.png    | Bin 0 -> 167 bytes
 toolkit/themes/winstripe/global/jar.mn        |   2 +
 .../winstripe/global/media/videocontrols.css  |  37 +++++-
 .../winstripe/global/media/volumeThumb.png    | Bin 0 -> 167 bytes
 8 files changed, 164 insertions(+), 21 deletions(-)
 create mode 100644 toolkit/themes/pinstripe/global/media/volumeThumb.png
 create mode 100644 toolkit/themes/winstripe/global/media/volumeThumb.png

diff --git a/toolkit/content/widgets/videocontrols.css b/toolkit/content/widgets/videocontrols.css
index 9bf173fb127..249d011add1 100644
--- a/toolkit/content/widgets/videocontrols.css
+++ b/toolkit/content/widgets/videocontrols.css
@@ -1,6 +1,6 @@
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
-.scrubber {
+.scrubber, .volumeControl {
     -moz-binding: url("chrome://global/content/bindings/videocontrols.xml#suppressChangeEvent");
 }
 
diff --git a/toolkit/content/widgets/videocontrols.xml b/toolkit/content/widgets/videocontrols.xml
index 523888df93b..88fe21617cc 100644
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -76,9 +76,13 @@
       <implementation>
 
       <field name="thumb">null</field>
+      <field name="type">null</field>
+      <field name="Utils">null</field>
       <constructor>
           <![CDATA[
           this.thumb = document.getAnonymousElementByAttribute(this, "class", "scale-thumb");
+          this.type = this.getAttribute("class");
+          this.Utils = document.getBindingParent(this.parentNode).Utils;
           ]]>
       </constructor>
 
@@ -94,7 +98,8 @@
             switch (which) {
               case "curpos":
                 // Update the time shown in the thumb.
-                this.thumb.setTime(newValue);
+                if (this.type == "scrubber")
+                    this.thumb.setTime(newValue);
 
                 // The value of userChanged is true when changing the position with the mouse,
                 // but not when pressing an arrow key. However, the base binding sets
@@ -102,7 +107,11 @@
                 if (!userChanged && !this._userChanged)
                   return;
                 this.setAttribute("value", newValue);
-                document.getBindingParent(this.parentNode).Utils.seekToPosition();
+
+                if (this.type == "scrubber")
+                    this.Utils.seekToPosition(newValue);
+                else if (this.type == "volumeControl")
+                    this.Utils.setVolume(newValue / 100);
                 break;
   
               case "minpos":
@@ -146,6 +155,10 @@
                         <label class="durationLabel"/>
                     </vbox>
                     <button class="muteButton" oncommand="document.getBindingParent(this).Utils.toggleMute();"/>
+                    <stack class="volumeStack" hidden="true">
+                        <box class="volumeBackgroundBar"/>
+                        <scale class="volumeControl" orient="vertical" dir="reverse" movetoclick="true"/>
+                    </stack>
                 </hbox>
             </vbox>
         </stack>
@@ -210,6 +223,8 @@
                 videocontrols : null,
                 playButton : null,
                 muteButton : null,
+                volumeStack    : null,
+                volumeControl  : null,
                 durationLabel  : null,
                 scrubberThumb  : null,
                 scrubber       : null,
@@ -250,6 +265,21 @@
                                     RUNTIME_STEP : 20
                                 },
 
+                // volumeFader holds the fade state for the volume <scale>.
+                volumeFader : {
+                                    name : "volume",
+                                    element : null,
+                                    maxSlide : null, // height when extended, set in init()
+                                    runtime : 0,
+                                    fadingIn : false,
+                                    isVisible : false,
+                                    timer : null,
+                                    delayTimer : null,
+                                    START_DELAY : 0,
+                                    RUNTIME_MAX : 200,
+                                    RUNTIME_STEP : 15
+                                },
+
                 firstFrameShown : false,
                 lastTimeUpdate : 0,
                 maxCurrentTimeSeen : 0,
@@ -268,6 +298,9 @@
                     this.playButton.setAttribute("paused", this.video.paused);
                     this.muteButton.setAttribute("muted", this.video.muted);
 
+                    var volume = this.video.muted ? 0 : Math.round(this.video.volume * 100);
+                    this.volumeControl.value = volume;
+
                     var duration = Math.round(this.video.duration * 1000); // in ms
                     var currentTime = Math.round(this.video.currentTime * 1000); // in ms
                     this.log("Initial playback position is at " + currentTime + " of " + duration);
@@ -342,7 +375,9 @@
                             this.playButton.setAttribute("paused", true);
                             break;
                         case "volumechange":
+                            var volume = this.video.muted ? 0 : Math.round(this.video.volume * 100);
                             this.muteButton.setAttribute("muted", this.video.muted);
+                            this.volumeControl.value = volume;
                             break;
                         case "loadedmetadata":
                             // If a <video> doesn't have any video data, treat it as <audio>
@@ -465,13 +500,18 @@
                     this.scrubber.pageIncrement = Math.round(duration / 10);
                 },
 
-                seekToPosition : function() {
-                    var newPosition = this.scrubber.getAttribute("value");
+                seekToPosition : function(newPosition) {
                     newPosition /= 1000; // convert from ms
                     this.log("+++ seeking to " + newPosition);
                     this.video.currentTime = newPosition;
                 },
 
+                setVolume : function(newVolume) {
+                    this.log("*** setting volume to " + newVolume);
+                    this.video.volume = newVolume;
+                    this.video.muted = false;
+                },
+
                 showPosition : function(currentTime, duration) {
                     // If the duration is unknown (because the server didn't provide
                     // it, or the video is a stream), then we want to fudge the duration
@@ -492,6 +532,15 @@
                     this.progressBar.value = Math.round(percent * 10000);
                 },
 
+                onVolumeMouseInOut : function (event) {
+                    // Ignore events caused by transitions between mute button and volumeStack,
+                    // or between nodes inside these two elements.
+                    if (this.isEventWithin(event, this.muteButton, this.volumeStack))
+                        return;
+                    var isMouseOver = (event.type == "mouseover");
+                    this.startFade(this.volumeFader, isMouseOver);
+                },
+
                 onMouseInOut : function (event) {
                     // If the controls are static, don't change anything.
                     if (!this.dynamicControls)
@@ -502,8 +551,7 @@
                     // size as the *content area* of the video element,
                     // but this is not the same as the video element's
                     // border area if the video has border or padding.
-                    if (this.isControlsOrDescendant(event.target) &&
-                        this.isControlsOrDescendant(event.relatedTarget))
+                    if (this.isEventWithin(event, this.videocontrols))
                         return;
 
                     var isMouseOver = (event.type == "mouseover");
@@ -596,12 +644,21 @@
                     else
                         opacity = Math.pow(1 - pos, 0.5);
                     fader.isVisible = (opacity ? true : false);
-                    //self.log("Fading " + fader.name + " to opacity " + opacity);
-
                     fader.element.style.opacity = opacity;
                     // Hide the element to ignore mouse clicks and reduce throbber CPU usage.
                     fader.element.setAttribute("hidden", !fader.isVisible);
 
+                    // If this fader also has a slide effect, change the CSS margin-top too.
+                    if (fader.maxSlide) {
+                        var marginTop;
+                        if (fader.fadingIn)
+                            marginTop = Math.round(fader.maxSlide * Math.pow(pos, 0.5));
+                        else
+                            marginTop = Math.round(fader.maxSlide * Math.pow(1 - pos, 0.5));
+
+                        fader.element.style.marginTop = marginTop;
+                    }
+
                     // Is the animation done?
                     if (pos == 1) {
                         clearInterval(fader.timer);
@@ -629,13 +686,16 @@
                     // controlling volume.
                 },
 
-                isControlsOrDescendant : function (node) {
-                    while (node) {
-                        if (node == this.videocontrols)
-                            return true;
-                        node = node.parentNode;
+                isEventWithin : function (event, parent1, parent2) {
+                    function isDescendant (node) {
+                        while (node) {
+                            if (node == parent1 || node == parent2)
+                                return true;
+                            node = node.parentNode;
+                        }
+                        return false;
                     }
-                    return false;
+                    return isDescendant(event.target) && isDescendant(event.relatedTarget);
                 },
 
                 log : function (msg) {
@@ -653,11 +713,15 @@
             this.Utils.videocontrols = this;
             this.Utils.isAudioOnly = (video instanceof HTMLAudioElement);
 
-            this.Utils.controlFader.element  = document.getAnonymousElementByAttribute(this, "class", "controlBar");
-            this.Utils.statusFader.element = document.getAnonymousElementByAttribute(this, "class", "statusOverlay");
+            this.Utils.controlFader.element = document.getAnonymousElementByAttribute(this, "class", "controlBar");
+            this.Utils.statusFader.element  = document.getAnonymousElementByAttribute(this, "class", "statusOverlay");
+            this.Utils.volumeFader.element  = document.getAnonymousElementByAttribute(this, "class", "volumeStack");
+
             this.Utils.statusIcon     = document.getAnonymousElementByAttribute(this, "class", "statusIcon");
             this.Utils.playButton     = document.getAnonymousElementByAttribute(this, "class", "playButton");
             this.Utils.muteButton     = document.getAnonymousElementByAttribute(this, "class", "muteButton");
+            this.Utils.volumeControl  = document.getAnonymousElementByAttribute(this, "class", "volumeControl");
+            this.Utils.volumeStack    = document.getAnonymousElementByAttribute(this, "class", "volumeStack");
             this.Utils.progressBar    = document.getAnonymousElementByAttribute(this, "class", "progressBar");
             this.Utils.bufferBar      = document.getAnonymousElementByAttribute(this, "class", "bufferBar");
             this.Utils.scrubber       = document.getAnonymousElementByAttribute(this, "class", "scrubber");
@@ -687,6 +751,17 @@
             for each (var event in this.Utils.videoEvents)
                 video.addEventListener(event, this.Utils, false);
 
+            // Determine the height of the volumeFader when extended (which is controlled by CSS).
+            // Its .clientHeight seems to be 0 here, so use the theme's initial value. (eg "-70px")
+            this.Utils.volumeFader.maxSlide = parseInt(window.getComputedStyle(this.Utils.volumeStack, null)
+                                                             .getPropertyValue("margin-top"));
+            var self = this;
+            function volumeBouncer(event) { self.Utils.onVolumeMouseInOut(event); }
+            this.Utils.muteButton.addEventListener("mouseover", volumeBouncer, false);
+            this.Utils.muteButton.addEventListener("mouseout", volumeBouncer, false);
+            this.Utils.volumeStack.addEventListener("mouseover", volumeBouncer, false);
+            this.Utils.volumeStack.addEventListener("mouseout", volumeBouncer, false);
+
             this.Utils.log("--- videocontrols initialized ---");
             ]]>
             </body>
diff --git a/toolkit/themes/pinstripe/global/jar.mn b/toolkit/themes/pinstripe/global/jar.mn
index b45b6fc460a..68cc3acd433 100644
--- a/toolkit/themes/pinstripe/global/jar.mn
+++ b/toolkit/themes/pinstripe/global/jar.mn
@@ -182,6 +182,7 @@ classic.jar:
 +  skin/classic/global/media/scrubberThumbWide.png                    (media/scrubberThumbWide.png)
 +  skin/classic/global/media/error.png                                (media/error.png)
 +  skin/classic/global/media/throbber.png                             (media/throbber.png)
++  skin/classic/global/media/volumeThumb.png                          (media/volumeThumb.png)
 +  skin/classic/global/menu/menu-arrow-dis.gif                        (menu/menu-arrow-dis.gif)
 +  skin/classic/global/menu/menu-arrow-hov.gif                        (menu/menu-arrow-hov.gif)
 +  skin/classic/global/menu/menu-arrow.gif                            (menu/menu-arrow.gif)
diff --git a/toolkit/themes/pinstripe/global/media/videocontrols.css b/toolkit/themes/pinstripe/global/media/videocontrols.css
index bdafea05602..5f6d693f0cd 100644
--- a/toolkit/themes/pinstripe/global/media/videocontrols.css
+++ b/toolkit/themes/pinstripe/global/media/videocontrols.css
@@ -33,6 +33,38 @@
     background: url(chrome://global/skin/media/unmuteButton.png) no-repeat center;
 }
 
+.volumeStack {
+    width: 28px;
+    height: 70px;
+    background-color: rgba(35,31,32,0.74);
+    /* use negative margin to place stack over the mute button to its left. */
+    margin: -70px 3px 28px -31px;
+    overflow: hidden; /* crop it when sliding down, don't grow the control bar */
+    position: relative; /* Trick to work around negative margin interfering with dragging the thumb. */
+    padding-top: 6px;
+}
+
+.volumeControl {
+    min-height: 64px;
+}
+
+/* .scale-thumb is an element inside the <scale> implementation. */
+.volumeControl .scale-thumb {
+    /* Override the default thumb appearance with a custom image. */
+    -moz-appearance: none;
+    background: url(chrome://global/skin/media/volumeThumb.png) no-repeat center;
+    border: none;
+    min-width: 16px;
+    min-height: 11px;
+}
+
+.volumeBackgroundBar {
+    /* margin left/right: make bar 8px wide (control width = 28, minus 2 * 10 margin) */
+    margin: 0px 10px 0px 10px;
+    background-color: rgba(255,255,255,0.5);
+    -moz-border-radius: 4px 4px;
+}
+
 .durationBox {
     -moz-box-pack: center;
 }
@@ -78,7 +110,7 @@
 }
 
 /* .scale-slider is an element inside the <scale> implementation. */
-.scale-slider {
+.scrubber .scale-slider, .volumeControl .scale-slider {
     /* Hide the default horizontal bar. */
     -moz-appearance: none;
     background: none;
@@ -86,7 +118,7 @@
 }
 
 /* .scale-thumb is an element inside the <scale> implementation. */
-.scale-thumb {
+.scrubber .scale-thumb {
     /* Override the default thumb appearance with a custom image. */
     -moz-appearance: none;
     background: transparent;
diff --git a/toolkit/themes/pinstripe/global/media/volumeThumb.png b/toolkit/themes/pinstripe/global/media/volumeThumb.png
new file mode 100644
index 0000000000000000000000000000000000000000..75ab7fb282ad3a867d64a03dba49694c0f319d26
GIT binary patch
literal 167
zcmeAS@N?(olHy`uVBq!ia0vp^0zk~o!3HGD9`RZNsXR{?#}JM4$q5Zj|Ns9tXZZN{
ze+#3BFEg_d1J{}V{#G;iFVA5R-dXYS(f>=AF8zP^?p=hT2j3DlPL&283;xSJ-pW>P
zMo(;*d#>KN;jx8LVXFayIm7(R8!K54GkC=~6iNh$af>WxUUDSF!Ox3@;o_V(bDqC`
R?FO`&!PC{xWt~$(69DdSJSG4D

literal 0
HcmV?d00001

diff --git a/toolkit/themes/winstripe/global/jar.mn b/toolkit/themes/winstripe/global/jar.mn
index 177b6f7c931..6ed6cee72d9 100644
--- a/toolkit/themes/winstripe/global/jar.mn
+++ b/toolkit/themes/winstripe/global/jar.mn
@@ -153,6 +153,7 @@ classic.jar:
         skin/classic/global/media/scrubberThumb.png              (media/scrubberThumb.png)
         skin/classic/global/media/scrubberThumbWide.png          (media/scrubberThumbWide.png)
         skin/classic/global/media/throbber.png                   (media/throbber.png)
+        skin/classic/global/media/volumeThumb.png                (media/volumeThumb.png)
         skin/classic/global/media/error.png                      (media/error.png)
         skin/classic/global/radio/radio-check.gif                (radio/radio-check.gif)
         skin/classic/global/radio/radio-check-dis.gif            (radio/radio-check-dis.gif)
@@ -327,6 +328,7 @@ classic.jar:
         skin/classic/aero/global/media/scrubberThumb.png                 (media/scrubberThumb.png)
         skin/classic/aero/global/media/scrubberThumbWide.png             (media/scrubberThumbWide.png)
         skin/classic/aero/global/media/throbber.png                      (media/throbber.png)
+        skin/classic/aero/global/media/volumeThumb.png                   (media/volumeThumb.png)
         skin/classic/aero/global/media/error.png                         (media/error.png)
         skin/classic/aero/global/radio/radio-check.gif                   (radio/radio-check.gif)
         skin/classic/aero/global/radio/radio-check-dis.gif               (radio/radio-check-dis.gif)
diff --git a/toolkit/themes/winstripe/global/media/videocontrols.css b/toolkit/themes/winstripe/global/media/videocontrols.css
index 53a0b04a8a0..91d66735b32 100644
--- a/toolkit/themes/winstripe/global/media/videocontrols.css
+++ b/toolkit/themes/winstripe/global/media/videocontrols.css
@@ -35,6 +35,39 @@
     background: url(chrome://global/skin/media/unmuteButton.png) no-repeat center;
 }
 
+.volumeStack {
+    width: 28px;
+    height: 70px;
+    background-color: rgba(35,31,32,0.74);
+    /* use negative margin to place stack over the mute button to its left. */
+    margin: -70px 3px 28px -31px;
+    overflow: hidden; /* crop it when sliding down, don't grow the control bar */
+    position: relative; /* Trick to work around negative margin interfering with dragging the thumb. */
+    padding-top: 6px;
+}
+
+.volumeControl {
+    min-height: 64px;
+}
+
+/* .scale-thumb is an element inside the <scale> implementation. */
+.volumeControl .scale-thumb {
+    /* Override the default thumb appearance with a custom image. */
+    -moz-appearance: none;
+    background: url(chrome://global/skin/media/volumeThumb.png) no-repeat center;
+    border: none;
+    min-width: 16px;
+    min-height: 11px;
+}
+
+.volumeBackgroundBar {
+    /* margin left/right: make bar 8px wide (control width = 28, minus 2 * 10 margin) */
+    margin: 0px 10px 0px 10px;
+    background-color: rgba(255,255,255,0.5);
+    -moz-border-radius: 4px 4px;
+}
+
+
 .durationBox {
     -moz-box-pack: center;
 }
@@ -85,7 +118,7 @@
 }
 
 /* .scale-slider is an element inside the <scale> implementation. */
-.scale-slider {
+.scrubber .scale-slider, .volumeControl .scale-slider {
     /* Hide the default horizontal bar. */
     -moz-appearance: none;
     background: none;
@@ -93,7 +126,7 @@
 }
 
 /* .scale-thumb is an element inside the <scale> implementation. */
-.scale-thumb {
+.scrubber .scale-thumb {
     /* Override the default thumb appearance with a custom image. */
     -moz-appearance: none;
     background: transparent;
diff --git a/toolkit/themes/winstripe/global/media/volumeThumb.png b/toolkit/themes/winstripe/global/media/volumeThumb.png
new file mode 100644
index 0000000000000000000000000000000000000000..75ab7fb282ad3a867d64a03dba49694c0f319d26
GIT binary patch
literal 167
zcmeAS@N?(olHy`uVBq!ia0vp^0zk~o!3HGD9`RZNsXR{?#}JM4$q5Zj|Ns9tXZZN{
ze+#3BFEg_d1J{}V{#G;iFVA5R-dXYS(f>=AF8zP^?p=hT2j3DlPL&283;xSJ-pW>P
zMo(;*d#>KN;jx8LVXFayIm7(R8!K54GkC=~6iNh$af>WxUUDSF!Ox3@;o_V(bDqC`
R?FO`&!PC{xWt~$(69DdSJSG4D

literal 0
HcmV?d00001