Bug 43628 - MathML mroot radical should use color defined by style
Summary: MathML mroot radical should use color defined by style
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: MathML (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.6
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-06 10:26 PDT by François Sausset
Modified: 2010-08-09 08:52 PDT (History)
1 user (show)

See Also:

Attachments
Patch (71.93 KB, patch)
2010-08-06 10:32 PDT, François Sausset 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description François Sausset 2010-08-06 10:26:47 PDT 
Drawing of the root radical should take into account the color defined in style.
Comment 1 François Sausset 2010-08-06 10:32:01 PDT 
Created attachment 63738 [details]
Patch
Comment 2 Eric Seidel (no email) 2010-08-06 13:13:44 PDT 
Comment on attachment 63738 [details]
Patch

Looks like all this does is adds a test.  The change from using sRGB to Device is not tested (I'm also not sure if it's correct or not.)
Comment 3 François Sausset 2010-08-06 13:21:57 PDT 
(In reply to comment #2)
> (From update of attachment 63738 [details])
> Looks like all this does is adds a test.  The change from using sRGB to Device is not tested (I'm also not sure if it's correct or not.)

Changing sRGB to Device makes the test work or not.

I admit that I don't really understand why colorSpace has an effect, but my knowledge of all the graphic context machinery is quite limited.
I tried to understand but did not manage to.
Comment 4 Eric Seidel (no email) 2010-08-09 08:52:26 PDT 
Comment on attachment 63738 [details]
Patch

Clearing flags on attachment: 63738

Committed r64984: <http://trac.webkit.org/changeset/64984>
Comment 5 Eric Seidel (no email) 2010-08-09 08:52:32 PDT 
All reviewed patches have been landed.  Closing bug.