[webkit-reviews] review granted: [Bug 86919] Don't force rendering in finishAllRendering : [Attachment 142816] Patch

[bugzilla-daemon at webkit.org]

James Robinson <jamesr at chromium.org> has granted Antoine Labour
<piman at chromium.org>'s request for review:
Bug 86919: Don't force rendering in finishAllRendering
https://bugs.webkit.org/show_bug.cgi?id=86919

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

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=142816&action=review


Yes!

>> Source/WebCore/ChangeLog:8
>> +	    No new tests. (OOPS!)
> 
> You should remove the 'No new tests' and either add and list tests, or
explain why no new tests were possible.  [changelog/nonewtests] [5]

You'll have to do this - might be simple enough to write a CCLayerTreeHostTest
that hits this case.