[webkit-reviews] review denied: [Bug 99815] [Shadow] ASSERT triggered when we try reprojecting fallback elements : [Attachment 169561] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 19 09:20:53 PDT 2012


Dimitri Glazkov (Google) <dglazkov at chromium.org> has denied Shinya Kawanaka
<shinyak at chromium.org>'s request for review:
Bug 99815: [Shadow] ASSERT triggered when we try reprojecting fallback elements
https://bugs.webkit.org/show_bug.cgi?id=99815

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

------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
I see what you're doing here, but as Dominic would say, the solution looks a
bit "skeezy" :)

1) Why is fallback content attached in the first place?
2) If it is, wouldn't it be more civil to detach and reattach it?


More information about the webkit-reviews mailing list