Bug 1289510 - Bookmark toolbar items should match navbar toolbar item hover style when lightweight themes are applied. r=jaws

MozReview-Commit-ID: IF8Qfi524Z7

--HG--
extra : rebase_source : a01517070371c2f68dbbf78388d280d7947eeacf

browser/themes/linux/browser.css

@@ -149,7 +149,7 @@ toolbarbutton.bookmark-item:not(.subviewbutton) {
 }
 
 toolbarbutton.bookmark-item:not(.subviewbutton):hover:not([open]) {
-  background-color: var(--toolbarbutton-hover-background);
+  background: var(--toolbarbutton-hover-background);
   border-color: var(--toolbarbutton-hover-bordercolor);
 }
 
@@ -160,6 +160,11 @@ toolbarbutton.bookmark-item[open="true"] {
   border-color: var(--toolbarbutton-active-bordercolor);
 }
 
+toolbarbutton.bookmark-item:not(.subviewbutton):hover:-moz-lwtheme {
+  background: var(--toolbarbutton-hover-background);
+  border-color: var(--toolbarbutton-hover-bordercolor);
+}
+
 .bookmark-item > .toolbarbutton-icon,
 #personal-bookmarks[cui-areatype="toolbar"] > #bookmarks-toolbar-placeholder > .toolbarbutton-icon {
   width: 16px;

browser/themes/windows/browser.css

@@ -466,6 +466,9 @@ toolbarbutton.bookmark-item:not(.subviewbutton),
   padding: 2px 3px;
   -moz-appearance: none;
   border: 1px solid transparent;
+  border-radius: 1px;
+  transition-property: background-color, border-color, box-shadow;
+  transition-duration: 150ms;
 }
 
 toolbarbutton.bookmark-item:not(.subviewbutton):hover:not([disabled="true"]):not([open]) {
@@ -480,6 +483,12 @@ toolbarbutton.bookmark-item[open="true"] {
   background: var(--toolbarbutton-active-background);
 }
 
+toolbarbutton.bookmark-item:not(.subviewbutton):hover:not([disabled="true"]):-moz-lwtheme {
+  background-color: rgba(255,255,255,.25);
+  background-origin: padding-box;
+  background-clip: padding-box;
+}
+
 .bookmark-item > .toolbarbutton-icon,
 #personal-bookmarks[cui-areatype="toolbar"] > #bookmarks-toolbar-placeholder > .toolbarbutton-icon {
   width: 16px;