[Webkit-unassigned] [Bug 113318] [GTK][AC] Support preserves3D css property for clutter ac backend.

[bugzilla-daemon at webkit.org]

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





--- Comment #13 from ChangSeok Oh <kevin.cs.oh at gmail.com>  2013-04-02 08:05:00 PST ---
(In reply to comment #12)
> (From update of attachment 196080 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=196080&action=review
> 
> cq- because of the suggestion
> 
> > Source/WebCore/platform/graphics/clutter/GraphicsLayerClutter.cpp:707
> > -        newSublayers.append(childLayer);
> > +        graphicsLayerActorSetFlatten(childLayer, !(preserves3D() || currentChild->preserves3D()));
> 
> > "If the preserve3D() of this layer returns true, itself and its children should not be flattened" is better to me.
> > But I don't mind removing the comment if it causes that you're confused. :p
> 
> The thing is that comment you suggest does not describe what the code is doing, do you agree with that? What the code does is set flatten to true on the child layer if both the current layer and the child do *not* preserve 3D. So I think you could say this:
> 
> "The child layer only preserves 3D if both itself and its parent have preserves3D set."
> 
> But feel free to omit the comment, I think the code speaks for itself.

kov, I think "The child layer only preserves 3D if either itself or its parent has preserves3D set." is correct, not *both*

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