[webkit-reviews] review denied: [Bug 102541] Add hasTouchEventhandlersAt to WebView API : [Attachment 175250] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Nov 25 21:59:09 PST 2012


James Robinson <jamesr at chromium.org> has denied yusufo at chromium.org's request
for review:
Bug 102541: Add hasTouchEventhandlersAt to WebView API
https://bugs.webkit.org/show_bug.cgi?id=102541

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

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=175250&action=review


The interface should move I think, but otherwise looks good.

> Source/WebKit/chromium/public/WebView.h:275
> +    // Check whether the given point hits any registered touch event
handlers.
> +    virtual bool hasTouchEventHandlersAt(const WebPoint&) = 0;

This should be on WebWidget - that's where handleInputEvent() is


More information about the webkit-reviews mailing list