[webkit-reviews] review granted: [Bug 23220] Roll layoutDelta into layoutState : [Attachment 26639] Patch, changelog

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 12 12:19:56 PST 2009


mitz at webkit.org has granted Simon Fraser (smfr) <simon.fraser at apple.com>'s
request for review:
Bug 23220: Roll layoutDelta into layoutState
https://bugs.webkit.org/show_bug.cgi?id=23220

Attachment 26639: Patch, changelog
https://bugs.webkit.org/attachment.cgi?id=26639&action=review

------- Additional Comments from mitz at webkit.org
> +    RenderView* v;
> +    if ((v = view()) && v->layoutStateEnabled()) {

Even though it increases the maximum indentation, I am not sure I like this
change, because it moves v outside the scope of the if statement.

r=me


More information about the webkit-reviews mailing list