Backout 64b58e1da898 (bug 664925) because of reftest oranges on OS X
authorEhsan Akhgari <ehsan@mozilla.com>
Mon, 20 Jun 2011 15:25:57 -0400
changeset 71406 ea1856ff64d133b229185108ea8254247a397da3
parent 71405 634f82bc05fb6a39ee2c82d28249696839ef0d20
child 71407 303ddaeb736f8bd3a9a31bb911b87a8d91502dfb
push id20539
push usermlamouri@mozilla.com
push dateTue, 21 Jun 2011 07:35:09 +0000
treeherdermozilla-central@bf714fffdfdf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs664925
milestone7.0a1
backs out64b58e1da898dc8f3f91ffe16cb66f10ed5ba669
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backout 64b58e1da898 (bug 664925) because of reftest oranges on OS X
widget/reftests/664925.xhtml
widget/reftests/reftest.list
widget/src/cocoa/nsNativeThemeCocoa.mm
deleted file mode 100644
--- a/widget/reftests/664925.xhtml
+++ /dev/null
@@ -1,1 +0,0 @@
-<html xmlns="http://www.w3.org/1999/xhtml"><div><td style="-moz-appearance: progressbar;"></td></div></html>
--- a/widget/reftests/reftest.list
+++ b/widget/reftests/reftest.list
@@ -1,3 +1,2 @@
 skip-if(!cocoaWidget) != 507947.html about:blank
 == progressbar-fallback-default-style.html progressbar-fallback-default-style-ref.html
-load 664925.xhtml
--- a/widget/src/cocoa/nsNativeThemeCocoa.mm
+++ b/widget/src/cocoa/nsNativeThemeCocoa.mm
@@ -1138,20 +1138,17 @@ nsNativeThemeCocoa::DrawFrame(CGContextR
 }
 
 static const CellRenderSettings progressSettings[2][2] = {
   // Vertical progress bar.
   {
     // Determined settings.
     {
       {
-        // TODO: for the moment, DrawCellWithSnapping doesn't handle NSZeroSize
-        // values in this array so we have to use negative values until
-        // bug 665571 is fixed.
-        NSMakeSize(-5, -5), // mini
+        NSZeroSize, // mini
         NSMakeSize(10, 0), // small
         NSMakeSize(16, 0)  // regular
       },
       {
         NSZeroSize, NSZeroSize, NSZeroSize
       },
       {
         { // Leopard
@@ -1159,20 +1156,17 @@ static const CellRenderSettings progress
           {1, 1, 1, 1},     // small
           {1, 1, 1, 1}      // regular
         }
       }
     },
     // There is no horizontal margin in regular undetermined size.
     {
       {
-        // TODO: for the moment, DrawCellWithSnapping doesn't handle NSZeroSize
-        // values in this array so we have to use negative values until
-        // bug 665571 is fixed.
-        NSMakeSize(-5, -5), // mini
+        NSZeroSize, // mini
         NSMakeSize(10, 0), // small
         NSMakeSize(16, 0)  // regular
       },
       {
         NSZeroSize, NSZeroSize, NSZeroSize
       },
       {
         { // Leopard