[webkit-reviews] review denied: [Bug 6503] content property doesn't support quotes : [Attachment 83029] Proposed Patch. Addressed DHyatt's concerns.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 1 11:58:00 PST 2011


Dave Hyatt <hyatt at apple.com> has denied  review:
Bug 6503: content property doesn't support quotes
https://bugs.webkit.org/show_bug.cgi?id=6503

Attachment 83029: Proposed Patch. Addressed DHyatt's concerns.
https://bugs.webkit.org/attachment.cgi?id=83029&action=review

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
Two gotchas:

(1) You need to patch RenderStyle::diff for when quotes change dynamically to
supply the appropriate hint (layout hint I think).
(2) It seems like:

&& *quotes.get() == *o.quotes.get()

will crash if you are comparing against objects with quotes:none, since you'll
deref 0.


More information about the webkit-reviews mailing list