Bug 388334 - Page Info theme should be more reusable by extensions.

patch by Florian Quèze <f.qu@queze.net>, r=mano

browser/base/content/pageinfo/pageInfo.js

@@ -375,13 +375,15 @@ function toggleGroupbox(id)
   var elt = document.getElementById(id);
   if (elt.hasAttribute("closed")) {
     elt.removeAttribute("closed");
-    if (id == "metaTags")
-      elt.flex = 1;
+    if (elt.flexWhenOpened)
+      elt.flex = elt.flexWhenOpened;
   }
   else {
     elt.setAttribute("closed", "true");
-    if (id == "metaTags")
+    if (elt.flex) {
+      elt.flexWhenOpened = elt.flex;
       elt.flex = 0;
+    }
   }
 }
 

browser/base/content/pageinfo/pageInfo.xul

@@ -178,7 +178,7 @@
         </rows>
       </grid>
       <separator class="thin"/>
-      <groupbox id="metaTags" flex="1">
+      <groupbox id="metaTags" flex="1" class="collapsable treebox">
         <caption id="metaTagsCaption" onclick="toggleGroupbox('metaTags');"/>
         <tree id="metatree" flex="1" hidecolumnpicker="true" contextmenu="picontext">
           <treecols>

browser/themes/pinstripe/browser/pageInfo.css

@@ -134,19 +134,16 @@ textbox[disabled] {
 }
 
 /* General Tab */
-.caption-icon {
+groupbox.collapsable caption .caption-icon {
   width: 11px;
   height: 11px;
   background-repeat: no-repeat;
   background-position: center;
   -moz-margin-end: 2px;
-}
-
-groupbox caption .caption-icon { 
   background-image: url("chrome://global/skin/arrow/arrow-dn.gif");
 }
 
-groupbox[closed="true"] caption .caption-icon { 
+groupbox.collapsable[closed="true"] caption .caption-icon {
   background-image: url("chrome://global/skin/arrow/arrow-rit.gif");
 }
 
@@ -155,7 +152,7 @@ groupbox tree {
   border: none;
 }
 
-#metaTags .groupbox-body { 
+groupbox.treebox .groupbox-body { 
   padding: 0;
 }
 
@@ -163,10 +160,6 @@ groupbox tree {
   -moz-margin-start: 10px;
 }
 
-#securityBox .caption-icon {
-  background-image: none;
-}
-
 #general-security-identity {
   white-space: -moz-pre-wrap;
   line-height: 2em;

browser/themes/winstripe/browser/pageInfo.css

@@ -136,28 +136,21 @@ textbox[disabled] {
 }
 
 /* General Tab */
-.caption-icon {
+groupbox.collapsable caption .caption-icon { 
   width: 9px;
   height: 9px;
   background-repeat: no-repeat;
   background-position: center;
   -moz-margin-start: 1px;
   -moz-margin-end: 3px;
-}
-
-#securityBoxCaption .caption-icon {
-  display: none;
-}
-
-groupbox caption .caption-icon { 
   background-image: url("chrome://global/skin/tree/twisty-open.png");
 }
 
-groupbox[closed="true"] {
+groupbox.collapsable[closed="true"] {
   border: none;
 }
 
-groupbox[closed="true"] caption .caption-icon { 
+groupbox.collapsable[closed="true"] caption .caption-icon { 
   background-image: url("chrome://global/skin/tree/twisty-clsd.png");
 }
 
@@ -166,7 +159,7 @@ groupbox tree {
   border: none;
 }
 
-#metaTags .groupbox-body { 
+groupbox.treebox .groupbox-body { 
   padding-top: 0;
 }
 
@@ -174,10 +167,6 @@ groupbox tree {
   -moz-margin-start: 10px;
 }
 
-#securityBox .caption-icon {
-  background-image: none;
-}
-
 #general-security-identity {
   white-space: -moz-pre-wrap;
   line-height: 2em;