[webkit-reviews] review denied: [Bug 83216] Send visually non-empty delegate callback consistently, when the page is restored from page cache. : [Attachment 137538] Patch

[bugzilla-daemon at webkit.org]

Brady Eidson <beidson at apple.com> has denied Zalan Bujtas <zbujtas at gmail.com>'s
request for review:
Bug 83216: Send visually non-empty delegate callback consistently, when the
page is restored from page cache.
https://bugs.webkit.org/show_bug.cgi?id=83216

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

------- Additional Comments from Brady Eidson <beidson at apple.com>
I have not looked at the contents of the patch closely.

I have a problem with the very notion of overloading "first visually non-empty
layout" for this when restoration from the page cache is not (necessarily)
layout.

What problem are you trying to solve, and can we solve it without making the
name "first visually non-empty layout" completely meaningless?