[webkit-reviews] review granted: [Bug 131071] willReveal edge events should be hooked up for overflow:scroll : [Attachment 228330] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 1 20:12:22 PDT 2014


Sam Weinig <sam at webkit.org> has granted Beth Dakin <bdakin at apple.com>'s request
for review:
Bug 131071: willReveal edge events should be hooked up for overflow:scroll
https://bugs.webkit.org/show_bug.cgi?id=131071

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

------- Additional Comments from Sam Weinig <sam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=228330&action=review


> Source/WebCore/dom/Document.cpp:5618
> +	// For each edge (top, bottom, left and right), send the will reveal
edge event for that direction

You have an extra space here.

> Source/WebCore/dom/Document.h:1177
> +    void sendWillRevealEdgeEventsIfNeeded(const IntPoint& oldPosition, const
IntPoint& newPosition, const IntRect& visibleRect, const IntSize& contentsSize,
Element* target = 0);

You should use Element* target = nullptr.  I would also rename this to
enqueueWillRevealEdgeEventsIfNeeded.  This might also be better on the
DocumentEventQueue class, not sure.


More information about the webkit-reviews mailing list