WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 47477
53269
http://www.webglearth.com/
is grayscale in Safari, color in Firefox
https://bugs.webkit.org/show_bug.cgi?id=53269
Summary
http://www.webglearth.com/ is grayscale in Safari, color in Firefox
Simon Fraser (smfr)
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Kenneth Russell
Comment 1
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.
Kenneth Russell
Comment 2
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.
Kenneth Russell
Comment 3
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
.
Chris Marrin
Comment 4
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?
Kenneth Russell
Comment 5
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.
Kenneth Russell
Comment 6
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
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug