Bug 53269 - http://www.webglearth.com/ is grayscale in Safari, color in Firefox
Summary: http://www.webglearth.com/ is grayscale in Safari, color in Firefox
Status: RESOLVED DUPLICATE of bug 47477
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Kenneth Russell
URL: http://www.webglearth.com/
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-27 17:06 PST by Simon Fraser (smfr)
Modified: 2011-01-28 18:09 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2011-01-27 17:06:52 PST
http://www.webglearth.com/ is grayscale on Mac, in WebKit. In Firefox it's in color. This seems like a bug.
Comment 1 Kenneth Russell 2011-01-27 17:49:22 PST
That's strange. It probably has something to do with which image loaders are being used. It appears in color in Chromium and grayscale in Safari. I'll take a closer look.
Comment 2 Kenneth Russell 2011-01-27 19:21:54 PST
The CGImageRef coming back from ImageSource::createFrameAtIndex (ImageSourceCG.cpp) thinks that it's 8 bits per pixel. I'm stumped so far why this is happening only for the images on this site. Most WebGL content using textures (including PNG textures like this site) displays properly.
Comment 3 Kenneth Russell 2011-01-27 19:47:19 PST
The issue is that the PNGs loaded by this application are indexed, and the CG image loading path for WebGL doesn't handle them properly. See https://bugs.webkit.org/show_bug.cgi?id=47477 .
Comment 4 Chris Marrin 2011-01-28 17:23:21 PST
So then shouldn't this be marked a dup of that? Or is there more to it than just handling indexed PNG correctly?
Comment 5 Kenneth Russell 2011-01-28 18:00:10 PST
(In reply to comment #4)
> So then shouldn't this be marked a dup of that? Or is there more to it than just handling indexed PNG correctly?

I'll close it as a dup shortly. Patch forthcoming for the other bug.
Comment 6 Kenneth Russell 2011-01-28 18:09:00 PST
The patch for https://bugs.webkit.org/show_bug.cgi?id=47477 , which is currently out for review, fixes this problem.

*** This bug has been marked as a duplicate of bug 47477 ***