Bug 24197
| Summary: | ColorCG.cpp: Windows version of cgColor() is ifdefed on !PLATFORM(MAC) which is true for Chrome on OS X | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jeremy Moskovich <playmobil> |
| Component: | Platform | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | ||
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Mac | ||
| OS: | OS X 10.5 | ||
Jeremy Moskovich
PLATFORM(MAC) isn't defined for Chrome, so on OS X we're picking up the Windows version of cgColor() from ColorCG.cpp rather than the Mac version from ColorMac.mm.
The define needs to be changed to either use !PLATFORM(DARWIN) or PLATFORM(WIN) so Chrome on OS X doesn't pick up the Windows version.
https://bugs.webkit.org/show_bug.cgi?id=23512 also bears mentioning in this context.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Jeremy Moskovich
Fixed in r41345
Mark Mentovai
Jeremy, why did we want cgColor ColorMac.mm for mcChrome instead of ColorCG.cpp? I just helped Pam out with this for the merge (she'll probably contact you separately about focusRingColor), but I don't understand why you wanted this change, it seems like it's going in the wrong direction.