Bug 604885 - backout 8d54b45a181e a=backout
authorChris Pearce <chris@pearce.org.nz>
Tue, 21 Dec 2010 07:45:02 +1300
changeset 59521 36abf00aa201446dec511eb691e4847d1ca7e348
parent 58177 8d54b45a181e67d878bca1d22055746f304e60ec
child 59522 a92cfed15419bb75893f1adcbb2808c2858cafc8
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
reviewersbackout
bugs604885
milestone2.0b8pre
Bug 604885 - backout 8d54b45a181e a=backout
toolkit/content/widgets/videocontrols.xml
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -713,25 +713,23 @@
                     if (element.className == "controlBar" && fadeIn && element.hidden)
                         this.scrubber.valueChanged("curpos", this.video.currentTime * 1000, false);
 
                     if (immediate)
                         element.setAttribute("immediate", true);
                     else
                         element.removeAttribute("immediate");
 
-                    // We must force style resolution (by accessing element.clientTop)
-                    // in order to ensure our size changes correctly when the status
-                    // overlay hides and shows.
                     if (fadeIn) {
                         element.setAttribute("hidden", false);
-                        element.clientTop;
+                        // force style resolution, so that transition begins
+                        // when we remove the attribute.
+                        getComputedStyle(element, "").display;
                         element.removeAttribute("fadeout");
                     } else {
-                        element.clientTop;
                         element.setAttribute("fadeout", true);
                     }
                 },
 
                 onTransitionEnd : function (event) {
                     // Ignore events for things other than opacity changes.
                     if (event.propertyName != "opacity")
                         return;