[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 #7 from Naoki Takano <takano.naoki at gmail.com>  2011-02-22 00:30:09 PST ---
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])
> 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.