[Webkit-unassigned] [Bug 98700] New: ScrollingCoordinator is a hot mess of if-defs

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=98700

           Summary: ScrollingCoordinator is a hot mess of if-defs
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: bdakin at apple.com


ScrollingCoordinator is a really confusing class because of all of the if-defs and because of ScrollingCoordinatorNone and because of platform implementations. It is very difficult to add or change API to this class as a result.

I propose that we make ScrollingCoordinator just be a typedef that defines ScrollingCoordinatorMac to be ScrollingCoordinator, and same for Chromium and other platforms. Any shared functionality can be in something called ScrollingCoordinatorBase which all of the platform implementations will inherit from.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.