Snow Leopard packaged unittests failing test_platform_colors.xul

RESOLVED FIXED

Status

()

Core
Widget: Cocoa
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: lsblakk, Unassigned)

Tracking

Trunk
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Consistent failure:

10056 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color highlighttext is wrong: rgb(254, 255, 254)
testing color inactiveborder

10062 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color menu is wrong: rgb(254, 255, 254)
testing color menutext

10099 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color -moz-menuhovertext is wrong: rgb(254, 255, 254)
testing color -moz-menubartext

Comment 1

8 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1270240441.1270241786.26993.gz
Rev3 MacOSX Snow Leopard 10.6.2 mozilla-central debug test mochitest-other [testfailed] Started 12:51, finished 13:18
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1270264762.1270266132.24040.gz
Rev3 MacOSX Snow Leopard 10.6.2 mozilla-central debug test mochitest-other on 2010/04/02 20:19:22
s: talos-r3-snow-018
10076 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color highlighttext is wrong: rgb(254, 255, 254)
10082 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color menu is wrong: rgb(254, 255, 254)
10119 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_platform_colors.xul | platform color -moz-menuhovertext is wrong: rgb(254, 255, 254)
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1270530411.1270531167.9714.gz
Rev3 MacOSX Snow Leopard 10.6.2 mozilla-central opt test mochitest-other on 2010/04/05 22:06:51
s: talos-r3-snow-018
I marked the colors that were showing up as legal:
http://hg.mozilla.org/mozilla-central/rev/83491775950c

It would be good if somebody checks that that makes sense.

I'm a bit suspicious of the numbers behaving the way they are, though... could there be some sort of rounding bug in our interaction with color management or something?
(In reply to comment #7)
> I marked the colors that were showing up as legal:
> http://hg.mozilla.org/mozilla-central/rev/83491775950c

Thanks!

> It would be good if somebody checks that that makes sense.

I think it does. If those colors are the ones that the system gives us, they're by definition correct.

> I'm a bit suspicious of the numbers behaving the way they are, though... could
> there be some sort of rounding bug in our interaction with color management or
> something?

Hmm. The code is:

>   NSColor* deviceColor = [aColor colorUsingColorSpaceName:NSDeviceRGBColorSpace];
>   return NS_RGB((unsigned int)([deviceColor redComponent] * 255.0),
>                 (unsigned int)([deviceColor greenComponent] * 255.0),
>                 (unsigned int)([deviceColor blueComponent] * 255.0));

We should check whether using rounding makes any difference.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Updated

8 years ago
Blocks: 558777

Updated

8 years ago
No longer blocks: 548768
You need to log in before you can comment on or make changes to this bug.