[webkit-reviews] review denied: [Bug 70766] LayerRendererChromium::setVisible called needlessly without a visibility change : [Attachment 112253] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 24 15:19:39 PDT 2011


Kenneth Russell <kbr at google.com> has denied Michal Mocny
<mmocny at chromium.org>'s request for review:
Bug 70766: LayerRendererChromium::setVisible called needlessly without a
visibility change
https://bugs.webkit.org/show_bug.cgi?id=70766

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

------- Additional Comments from Kenneth Russell <kbr at google.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=112253&action=review


Nat's question is a good one. Small issue in the ChangeLog needs to be fixed as
well.

> Source/WebCore/ChangeLog:10
> +	   No new tests. (OOPS!)

This second OOPS will have to be removed in order for the commit queue to
process this patch.


More information about the webkit-reviews mailing list