[webkit-reviews] review requested: [Bug 11850] Webarchive fails to save images referenced in CSS : [Attachment 26215] Patch v4

[bugzilla-daemon at webkit.org]

David Kilzer (ddkilzer) <ddkilzer at webkit.org> has asked Darin Adler
<darin at apple.com> for review:
Bug 11850: Webarchive fails to save images referenced in CSS
https://bugs.webkit.org/show_bug.cgi?id=11850

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

------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at webkit.org>
Changes since Patch v3:

- Reverted const-ness of StyleBase::stylesheet().
- Removed const-ness from all addSubresourceStyleURLs() methods.
- Replaced switch() statements with virtual methods on CSSRule and CSSValue
subclasses (OO design FTW!).
- Removed abuse of RefPtr<> in favor of raw pointers.
- Renamed max variables.
- Inlined null-checked variable declarations into if() statements.
- Called immediate superclass instead of
StyledElement::addSubresourceAttributeURLs().
- Reverted changes to CSSBorderImageValue.h to include Rect.h (although it only
works because each source file where it's included also includes Rect.h).
- Tweaked ProcessingInstruction::addSubresourceAttributeURLs() to call
StyleSheet::baseURL().