Bug 689387 - Video controls should use new mouseenter/mouseleave events. r=dolske

toolkit/content/widgets/videocontrols.xml

@@ -819,21 +819,15 @@
                 },
 
                 onMouseInOut : function (event) {
+                    if (this.isTouchControl)
+                        return;
+
                     // If the controls are static, don't change anything.
                     if (!this.dynamicControls)
                         return;
 
                     clearTimeout(this._hideControlsTimeout);
-
-                    // Ignore events caused by transitions between child nodes.
-                    // Note that the videocontrols element is the same
-                    // 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.isEventWithin(event, this.videocontrols))
-                        return;
-
-                    var isMouseOver = (event.type == "mouseover");
+                    var isMouseOver = (event.type == "mouseenter");
 
                     // Suppress fading out the controls until the video has rendered
                     // its first frame. But since autoplay videos start off with no
@@ -1280,7 +1274,8 @@
                     this.muteButton.addEventListener("command", function() { self.toggleMute(); }, false);
                     this.playButton.addEventListener("command", function() { self.togglePause(); }, false);
                     this.fullscreenButton.addEventListener("command", function() { self.toggleFullscreen(); }, false );
-
+                    this.video.addEventListener("mouseenter", function(e) { self.onMouseInOut(e) }, false, true);
+                    this.video.addEventListener("mouseleave", function(e) { self.onMouseInOut(e) }, false, true);
                     this.controlsSpacer.addEventListener("click", function spacerClickHandler(e) {
                       if (e.button != 0 || self.hasError())
                         return;
@@ -1318,14 +1313,6 @@
     </implementation>
 
     <handlers>
-        <handler event="mouseover">
-            if (!this.isTouchControl)
-                this.Utils.onMouseInOut(event);
-        </handler>
-        <handler event="mouseout">
-            if (!this.isTouchControl)
-                this.Utils.onMouseInOut(event);
-        </handler>
         <handler event="mousemove">
             if (!this.isTouchControl)
                 this.Utils.onMouseMove(event);