[Webkit-unassigned] [Bug 54921] [Chromium] Issue 60965: Layout test fast/canvas/setWidthResetAfterForcedRender.html is failing on GPU path

[bugzilla-daemon at webkit.org]

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





--- Comment #8 from Naoki Takano <takano.naoki at gmail.com>  2011-02-22 19:16:13 PST ---
James,

Could you give me your thought?

Thanks,

(In reply to comment #7)
> James,
> 
> Thank you for your review.
> 
> As you know, requiresLayer() function is called bye
> RenderBoxModelObject::styleDidChange()
> RenderObject::updateFillImages()
> 
> If the returned values are true, the rendering tree becomes different from the non-accelerated rendering trees. The canvas layer order is different from non-accelerated.
> 
> For example, in setWidthResetAfterForce, the canvas element becomes the same layer of the root. That causes the test failure. That is why I change this part.
> 
> I want to make sure the layer purpose. I thought this looked like the very special layer to overlay the rendered images in partial. Is this correct? Or is there another purpose for the layer?
> 
> Still I'm wondering why the requiresLayer() can change rendering tree order.
> 
> Thanks,
> 
> (In reply to comment #4)
> > (From update of attachment 83250 [details] [details])
> > This is wrong, accelerated 2d canvases also require a layer. Why do you think this helps on that test?

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