Bug 1145909 - Use an image instead of toolbarbutton for the reading list button. r=florian

browser/base/content/browser.xul

@@ -827,10 +827,10 @@
                        hidden="true"
                        tooltiptext="&pageReportIcon.tooltip;"
                        onclick="gPopupBlockerObserver.onReportButtonClick(event);"/>
-                <toolbarbutton id="readinglist-addremove-button"
-                               class="tabbable urlbar-icon"
-                               hidden="true"
-                               oncommand="ReadingListUI.togglePageByBrowser(gBrowser.selectedBrowser);"/>
+                <image id="readinglist-addremove-button"
+                       class="urlbar-icon"
+                       hidden="true"
+                       onclick="ReadingListUI.togglePageByBrowser(gBrowser.selectedBrowser);"/>
                 <image id="reader-mode-button"
                        class="urlbar-icon"
                        hidden="true"

browser/themes/linux/browser.css

@@ -1630,10 +1630,6 @@ richlistitem[type~="action"][actiontype="switchtab"] > .ac-url-box > .ac-action-
 
 %include ../shared/readinglist/readinglist.inc.css
 
-#readinglist-addremove-button {
-  padding: 0 2px;
-}
-
 /* Reader mode button */
 
 #reader-mode-button {

browser/themes/osx/browser.css

@@ -2529,12 +2529,6 @@ richlistitem[type~="action"][actiontype="switchtab"][selected="true"] > .ac-url-
 
 %include ../shared/readinglist/readinglist.inc.css
 
-#readinglist-addremove-button {
-  padding: 3px;
-  -moz-padding-start: 2px;
-  -moz-padding-end: 1px;
-}
-
 /* Reader mode button */
 
 #reader-mode-button {

browser/themes/shared/readinglist/readinglist.inc.css

@@ -5,18 +5,8 @@
 }
 
 #readinglist-addremove-button {
-  -moz-appearance: none;
-  border: none;
   list-style-image: url("chrome://browser/skin/readinglist/icons.svg#addpage");
-}
-
-#readinglist-addremove-button:hover {
-  border: none;
-}
-
-#readinglist-addremove-button > .toolbarbutton-icon {
-  width: 14px;
-  height: 14px
+  -moz-image-region: rect(0, 14px, 14px, 0);
 }
 
 #readinglist-addremove-button:hover {

browser/themes/windows/browser.css

@@ -1578,10 +1578,6 @@ richlistitem[type~="action"][actiontype="switchtab"] > .ac-url-box > .ac-action-
 
 %include ../shared/readinglist/readinglist.inc.css
 
-#readinglist-addremove-button {
-  padding: 0 2px;
-}
-
 /* Reader mode button */
 
 #reader-mode-button {