[Webkit-unassigned] [Bug 27841] Fix box shadow painting: should pass shadow's color

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27841





--- Comment #8 from Crystal Zhang <crystal.zhang at torchmobile.com>  2009-07-30 13:03:16 PDT ---
(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > (In reply to comment #4)
> > > > (In reply to comment #3)
> > > > > (From update of attachment 33791 [details] [details] [details] [details] [details])
> > > > > I think this is wrong. What is the issue this is supposed to address?
> > > > 
> > > > It always uses Color::black to paint box shadow regardless shadow style, which
> > > > is wrong.
> > > 
> > > The code uses a black fill for the shape casting the shadow. The shadow
> > > properties, including its color, are based on the ShadowData structure from the
> > > style:
> > > 
> > >     for (ShadowData* shadow = s->boxShadow(); shadow; shadow = shadow->next) {
> > > […]
> > >         Color& shadowColor = shadow->color;
> > > […]
> > >             context->setShadow(shadowOffset, shadowBlur, shadowColor);
> > 
> > Thanks for your explain. Probably I misunderstood. One thing still confusing:
> > can I know the reason that it uses shadow color later in that function?
> 
> I am not sure which use of shadow color you are asking about. There are two
> branches, one for “normal” shadows and one for inset shadows, and each branch
> has its own call to setShadow() where it passes the shadow color.

The branch for inset shadows, it calls fillRoundedRect and fillRect with shadow
color.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.