[Webkit-unassigned] [Bug 59753] Chromium Mac: Use ScrollAnimatorMac.mm for overlay scrollbar support
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun May 1 13:27:44 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=59753
Sam Weinig <sam at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #91733|review? |review-
Flag| |
--- Comment #12 from Sam Weinig <sam at webkit.org> 2011-05-01 13:27:44 PST ---
(From update of attachment 91733)
View in context: https://bugs.webkit.org/attachment.cgi?id=91733&action=review
> Source/WebCore/platform/mac/ScrollAnimatorMac.h:29
> +#if ENABLE(SMOOTH_SCROLLING) || (PLATFORM(CHROMIUM) && OS(DARWIN))
Why doesn't Chromium on Mac just enable SMOOTH_SCROLLING if you want to take this code path. Adding additional defines defeats the purpose of have fine grained enabled macros.
> Source/WebCore/platform/mac/ScrollAnimatorMac.mm:54
> +static NSPoint IntPointToNSPoint(WebCore::IntPoint point) {
> + return NSMakePoint(point.x(), point.y());
> +}
> +
> +static WebCore::IntPoint NSPointToIntPoint(NSPoint point) {
> + return WebCore::IntPoint(point.x, point.y);
> +}
{ needs to be on the next line. Function can't start with a capital.
> Source/WebCore/platform/mac/ScrollAnimatorMac.mm:221
> + return IntPointToNSPoint(_animator->scrollableArea()->currentMousePosition());
Why is this necessary, IntPoint has a conversion to NSPoint.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list