[webkit-reviews] review granted: [Bug 16136] Use shared PixelDumpSupport for Mac DRT : [Attachment 17532] patch

[bugzilla-daemon at webkit.org]

Adam Roben <aroben at apple.com> has granted Sam Weinig <sam at webkit.org>'s request
for review:
Bug 16136: Use shared PixelDumpSupport for Mac DRT
http://bugs.webkit.org/show_bug.cgi?id=16136

Attachment 17532: patch
http://bugs.webkit.org/attachment.cgi?id=17532&action=edit

------- Additional Comments from Adam Roben <aroben at apple.com>
You might mention in the ChangeLog that you're now using ImageDiffCG as well.

+void drawWebViewIntoContext(CGContextRef);
+void repaintWithVerticalSweep(CGContextRef);
+void repaintWithHorizontalSweep(CGContextRef);
+void drawSelectionRectIntoContext(CGContextRef, CGRect);

How about:

void paintWebView(CGContextRef);
void repaintWebView(CGContextRef, bool horizontal);
void drawSelectionRect(CGContextRef, const CGRect&);

r=me