[webkit-reviews] review denied: [Bug 17897] Not Rendering Images Imported from XHTML Document : [Attachment 23003] Updated patch: tackles Eric's comments

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Aug 28 13:24:58 PDT 2008


Eric Seidel <eric at webkit.org> has denied Julien Chaffraix
<jchaffraix at webkit.org>'s request for review:
Bug 17897: Not Rendering Images Imported from XHTML Document
https://bugs.webkit.org/show_bug.cgi?id=17897

Attachment 23003: Updated patch: tackles Eric's comments
https://bugs.webkit.org/attachment.cgi?id=23003&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
I'm still slightly concerned by this patch as it might break existing callers
of updateFromElement(), or future callers who don't know about this "don't do
anything if I've ever seen an error" behavior.	One way to address that would
be to have updateFromElement take a flag which was used to trigger this kind of
clear.	Another would be to ASSERT in updateElement that the current source was
the same as the previous source, or simply to only not re-try in
updateFromElement when the current source was the same and there was already an
error.	Another way would be to just give updateFromElement a better name! (the
current one is bad because it's so generic)

Hum... I just still think we can make this API clearer.


More information about the webkit-reviews mailing list