Bug 1354612 - Remove synchronous style flush since the performance cost of it is not worth the minor bug it is fixing. r?gijs draft
authorJared Wein <jwein@mozilla.com>
Fri, 07 Apr 2017 13:43:23 -0400
changeset 558422 2f33b3e877aa45a052efdae5c4ab0e1589ecc731
parent 558406 22df0c5cfbf8a2aa2822bf9c763e6c1c5a3f69de
child 559909 c4f2f628890440f68faad50d56ace7b1986b89e8
push id52891
push userbmo:jaws@mozilla.com
push dateFri, 07 Apr 2017 17:44:34 +0000
reviewersgijs
bugs1354612
milestone55.0a1
Bug 1354612 - Remove synchronous style flush since the performance cost of it is not worth the minor bug it is fixing. r?gijs MozReview-Commit-ID: EYwek0iIFst
browser/base/content/tabbrowser.xml
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -5489,22 +5489,16 @@
             modifiedAttrs.push("soundplaying-scheduledremoval");
           }
 
           if (!tab.hasAttribute("soundplaying")) {
             tab.setAttribute("soundplaying", true);
             modifiedAttrs.push("soundplaying");
           }
 
-          if (modifiedAttrs.length) {
-            // Flush style so that the opacity takes effect immediately, in
-            // case the media is stopped before the style flushes naturally.
-            getComputedStyle(tab).opacity;
-          }
-
           this._tabAttrModified(tab, modifiedAttrs);
         ]]>
       </handler>
       <handler event="DOMAudioPlaybackStopped">
         <![CDATA[
           var tab = this.getTabFromAudioEvent(event)
           if (!tab) {
             return;