Bug 1410561: Remove frontend CSS hackarounds for gecko assertion-failure that should be fixed now. r=jaws

The Gecko assertion was fixed in bug 1379332.

MozReview-Commit-ID: 6UZKxELJNIX

--HG--
extra : rebase_source : e1e9b958c89053fc5758479023cab277b78af083

browser/extensions/pocket/skin/shared/pocket.css

@@ -44,14 +44,8 @@
   /* Since .toolbarbutton-icon uses a different width than the animatable box,
      we need to set a padding relative to the difference in widths. */
   margin-inline-start: calc((16px + 2 * var(--toolbarbutton-inner-padding) - 20px) / 2);
-  /* Set the min- and max- width and height of the box equal to that
-     of each frame of the SVG sprite. Setting the width and height via
-     the `width` and `height` CSS properties causes an assertion for
-     `inline-size less than zero: 'aContainingBlockISize >= 0'` (bug 1379332). */
-  min-width: 20px;
-  max-width: 20px;
-  min-height: 16px;
-  max-height: 16px;
+  width: 20px; /* Width of each frame within the SVG sprite */
+  height: 16px; /* Height of each frame within the SVG sprite */
 }
 
 #pocket-button-box[animate="true"] > #pocket-animatable-box {

browser/themes/shared/downloads/indicator.inc.css

@@ -168,12 +168,8 @@
 /* download start animation */
 
 #downloads-notification-anchor {
-  width: 42px;
-  /* Height is equal to height of each frame in the SVG animation
-     Use of min/max-height instead of height is to avoid an
-     assertion for `inline-size less than zero..` (bug 1379332). */
-  min-height: 98px;
-  max-height: 98px;
+  width: 42px; /* Width of each frame within the SVG sprite */
+  height: 98px; /* Height of each frame within the SVG sprite */
   overflow: hidden;
   /* animation is not directional and shouldn't be reversed in RTL */
   direction: ltr;

browser/themes/shared/toolbarbutton-icons.inc.css

@@ -105,14 +105,8 @@ toolbar[brighttext] {
   /* Since .toolbarbutton-icon uses a different width than the animatable-box,
      we need to set a padding relative to the difference in widths. */
   margin-inline-start: calc((16px + 2 * var(--toolbarbutton-inner-padding) - 18px) / 2);
-  /* Set the min- and max- width and height of the box equal to that
-     of each frame of the SVG sprite. Setting the width and height via
-     the `width` and `height` CSS properties causes an assertion for
-     `inline-size less than zero: 'aContainingBlockISize >= 0'` (bug 1379332). */
-  min-width: 18px;
-  max-width: 18px;
-  min-height: 20px;
-  max-height: 20px;
+  width: 18px; /* Width of each frame within the SVG sprite */
+  height: 20px; /* Height of each frame within the SVG sprite */
 }
 
 #reload-button > .toolbarbutton-animatable-box > .toolbarbutton-animatable-image,
@@ -480,12 +474,8 @@ toolbar[brighttext] {
      and the .toolbar-animatable-box. This is correct even in RTL because the item
      is positioned using `left`. */
   left: calc(var(--library-icon-x) + (16px + 2 * var(--toolbarbutton-inner-padding) - 22px) / 2);
-  /* Set the min- and max- width and height of the box equal to that
-     of each frame of the SVG sprite (must use min- and max- due to bug 1379332). */
-  min-width: 22px;
-  max-width: 22px;
-  min-height: 54px;
-  max-height: 54px;
+  width: 22px; /* Width of each frame within the SVG sprite */
+  height: 54px; /* Height of each frame within the SVG sprite */
   z-index: 2;
 }
 

browser/themes/shared/urlbar-searchbar.inc.css

@@ -299,12 +299,8 @@
   /* .urlbar-icon has width 28px. Each frame is 33px wide. Set margin-inline-start
      to be half the difference, -2.5px. */
   margin-inline-start: -2.5px;
-  /* Set the height to equal the height of each frame of the SVG. Must use
-     min- and max- width and height due to bug 1379332. */
-  min-width: 33px;
-  max-width: 33px;
-  min-height: 33px;
-  max-height: 33px;
+  width: 33px; /* Width of each frame within the SVG sprite */
+  height: 33px; /* Height of each frame within the SVG sprite */
 }
 
 :root[uidensity=compact] #star-button-box[animationsenabled] > #star-button[starred][animate] + #star-button-animatable-box {