Bug 1458375 - Remove empty binding attached to scrollbars. r?timdream draft
authorMike Conley <mconley@mozilla.com>
Tue, 29 May 2018 20:08:41 -0400
changeset 806912 47139835a9d7e053a5c126a91a856a31c4803ddd
parent 806911 92ec32795505a3aede71010ea7e93b1d49800eb2
child 806913 87caa0531fc0c68dca14f1b681b8e452ab1681e6
push id112986
push usermconley@mozilla.com
push dateTue, 12 Jun 2018 19:30:14 +0000
reviewerstimdream
bugs1458375
milestone62.0a1
Bug 1458375 - Remove empty binding attached to scrollbars. r?timdream MozReview-Commit-ID: HD6OHIVIhrq
toolkit/content/widgets/general.xml
toolkit/themes/osx/global/nativescrollbars.css
toolkit/themes/windows/global/xulscrollbars.css
--- a/toolkit/content/widgets/general.xml
+++ b/toolkit/content/widgets/general.xml
@@ -4,20 +4,16 @@
    - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
 
 
 <bindings id="generalBindings"
    xmlns="http://www.mozilla.org/xbl"
    xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
    xmlns:xbl="http://www.mozilla.org/xbl">
 
-  <!-- This binding is needed on <scrollbar> to prevent content process
-       startup performance regression; to be removed in bug 1458375 -->
-  <binding id="empty"/>
-
   <binding id="basecontrol">
     <implementation implements="nsIDOMXULControlElement">
       <!-- public implementation -->
       <property name="disabled" onset="if (val) this.setAttribute('disabled', 'true');
                                        else this.removeAttribute('disabled');
                                        return val;"
                                 onget="return this.getAttribute('disabled') == 'true';"/>
       <property name="tabIndex" onget="return parseInt(this.getAttribute('tabindex')) || 0"
--- a/toolkit/themes/osx/global/nativescrollbars.css
+++ b/toolkit/themes/osx/global/nativescrollbars.css
@@ -2,17 +2,16 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 @namespace html url("http://www.w3.org/1999/xhtml");
 
 scrollbar {
   -moz-appearance: scrollbar;
-  -moz-binding: url("chrome://global/content/bindings/general.xml#empty");
   cursor: default;
   background-color: white;
 }
 
 scrollbar[root="true"] {
   position: relative;
   z-index: 2147483647; /* largest positive value of a signed 32-bit integer */
 }
--- a/toolkit/themes/windows/global/xulscrollbars.css
+++ b/toolkit/themes/windows/global/xulscrollbars.css
@@ -8,17 +8,16 @@
 
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 @namespace html url("http://www.w3.org/1999/xhtml"); /* namespace for HTML elements */
 
 /* ::::: scrollbar ::::: */
 
 scrollbar {
   -moz-appearance: scrollbar-horizontal;
-  -moz-binding: url("chrome://global/content/bindings/general.xml#empty");
   cursor: default;
 }
 
 @media all and (-moz-overlay-scrollbars) {
   scrollbar[root="true"] {
     position: relative;
     z-index: 2147483647; /* largest positive value of a signed 32-bit integer */
   }