Bug 1463320 - Remove the "wizard-base" binding and import the "wizard.css" file as a document stylesheet. r=bgrins draft
authorPaolo Amadini <paolo.mozmail@amadzone.org>
Fri, 06 Jul 2018 15:51:41 +0100
changeset 815001 74bac0f0aa8169588c7e90666f803782d6d1ee71
parent 815000 6dd177179b7e1b854019f769a4d4cb148e1a16fd
child 815006 fbb16131c711725c6b1d64b099b434dfc8186bf3
push id115398
push userpaolo.mozmail@amadzone.org
push dateFri, 06 Jul 2018 14:52:32 +0000
reviewersbgrins
bugs1463320
milestone63.0a1
Bug 1463320 - Remove the "wizard-base" binding and import the "wizard.css" file as a document stylesheet. r=bgrins MozReview-Commit-ID: JRw6jnAhql2
toolkit/content/widgets.css
toolkit/content/widgets/wizard.xml
toolkit/mozapps/update/content/updates.xml
--- a/toolkit/content/widgets.css
+++ b/toolkit/content/widgets.css
@@ -15,8 +15,9 @@
 @import url("chrome://global/skin/menu.css");
 @import url("chrome://global/skin/menulist.css");
 @import url("chrome://global/skin/notification.css");
 @import url("chrome://global/skin/popup.css");
 @import url("chrome://global/skin/richlistbox.css");
 @import url("chrome://global/skin/splitter.css");
 @import url("chrome://global/skin/tabbox.css");
 @import url("chrome://global/skin/toolbar.css");
+@import url("chrome://global/skin/wizard.css");
--- a/toolkit/content/widgets/wizard.xml
+++ b/toolkit/content/widgets/wizard.xml
@@ -9,26 +9,17 @@
   %wizardDTD;
 ]>
 
 <bindings id="wizardBindings"
    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">
 
-  <binding id="wizard-base">
-    <resources>
-      <stylesheet src="chrome://global/skin/wizard.css"/>
-    </resources>
-  </binding>
-
   <binding id="wizard">
-    <resources>
-      <stylesheet src="chrome://global/skin/wizard.css"/>
-    </resources>
     <content>
       <xul:hbox class="wizard-header" anonid="Header"/>
 
       <xul:deck class="wizard-page-box" flex="1" anonid="Deck">
         <children includes="wizardpage"/>
       </xul:deck>
       <children/>
 
@@ -446,48 +437,48 @@
                group="system" action="this._hitEnter(event)"/>
       <handler event="keypress" keycode="VK_ESCAPE" group="system">
         if (!event.defaultPrevented)
           this.cancel();
       </handler>
     </handlers>
   </binding>
 
-  <binding id="wizardpage" extends="chrome://global/content/bindings/wizard.xml#wizard-base">
+  <binding id="wizardpage">
     <implementation>
       <field name="pageIndex">-1</field>
 
       <property name="pageid" onget="return this.getAttribute('pageid');"
                               onset="this.setAttribute('pageid', val);"/>
 
       <property name="next"   onget="return this.getAttribute('next');"
                               onset="this.setAttribute('next', val);
                                      this.parentNode._accessMethod = 'random';
                                      return val;"/>
     </implementation>
   </binding>
 
 #ifdef XP_MACOSX
-  <binding id="wizard-header" extends="chrome://global/content/bindings/wizard.xml#wizard-base">
+  <binding id="wizard-header">
     <content>
       <xul:stack class="wizard-header-stack" flex="1">
         <xul:vbox class="wizard-header-box-1">
           <xul:vbox class="wizard-header-box-text">
             <xul:label class="wizard-header-label" xbl:inherits="xbl:text=label"/>
           </xul:vbox>
         </xul:vbox>
         <xul:hbox class="wizard-header-box-icon">
           <xul:spacer flex="1"/>
           <xul:image class="wizard-header-icon" xbl:inherits="src=iconsrc"/>
         </xul:hbox>
       </xul:stack>
     </content>
   </binding>
 
-  <binding id="wizard-buttons" extends="chrome://global/content/bindings/wizard.xml#wizard-base">
+  <binding id="wizard-buttons">
     <content>
       <xul:vbox flex="1">
         <xul:hbox class="wizard-buttons-btm">
           <xul:button class="wizard-button" dlgtype="extra1" hidden="true"/>
           <xul:button class="wizard-button" dlgtype="extra2" hidden="true"/>
           <xul:button label="&button-cancel-mac.label;" class="wizard-button" dlgtype="cancel"/>
           <xul:spacer flex="1"/>
           <xul:button label="&button-back-mac.label;" accesskey="&button-back-mac.accesskey;"
@@ -508,29 +499,29 @@
         ]]></body>
       </method>
     </implementation>
 
   </binding>
 
 #else
 
-  <binding id="wizard-header" extends="chrome://global/content/bindings/wizard.xml#wizard-base">
+  <binding id="wizard-header">
     <content>
       <xul:hbox class="wizard-header-box-1" flex="1">
         <xul:vbox class="wizard-header-box-text" flex="1">
           <xul:label class="wizard-header-label" xbl:inherits="xbl:text=label"/>
           <xul:label class="wizard-header-description" xbl:inherits="xbl:text=description"/>
         </xul:vbox>
         <xul:image class="wizard-header-icon" xbl:inherits="src=iconsrc"/>
       </xul:hbox>
     </content>
   </binding>
 
-  <binding id="wizard-buttons" extends="chrome://global/content/bindings/wizard.xml#wizard-base">
+  <binding id="wizard-buttons">
     <content>
       <xul:vbox class="wizard-buttons-box-1" flex="1">
         <xul:separator class="wizard-buttons-separator groove"/>
         <xul:hbox class="wizard-buttons-box-2">
           <xul:button class="wizard-button" dlgtype="extra1" hidden="true"/>
           <xul:button class="wizard-button" dlgtype="extra2" hidden="true"/>
           <xul:spacer flex="1" anonid="spacer"/>
 #ifdef XP_UNIX
--- a/toolkit/mozapps/update/content/updates.xml
+++ b/toolkit/mozapps/update/content/updates.xml
@@ -7,19 +7,16 @@
 <!DOCTYPE bindings SYSTEM "chrome://mozapps/locale/update/updates.dtd">
 
 <bindings id="updatesBindings"
           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">
 
   <binding id="updateheader" extends="chrome://global/content/bindings/wizard.xml#wizard-header">
-    <resources>
-      <stylesheet src="chrome://global/skin/wizard.css"/>
-    </resources>
     <content>
       <xul:hbox class="wizard-header update-header" flex="1">
         <xul:vbox class="wizard-header-box-1">
           <xul:vbox class="wizard-header-box-text">
             <xul:label class="wizard-header-label" xbl:inherits="value=label"/>
           </xul:vbox>
         </xul:vbox>
       </xul:hbox>