[Webkit-unassigned] [Bug 122958] New: [WK2] REGRESSION(r156472): scrolling coordinator is always enabled incorrectly.

[bugzilla-daemon at webkit.org]

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

           Summary: [WK2] REGRESSION(r156472): scrolling coordinator is
                    always enabled incorrectly.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: h047679 at stud.u-szeged.hu
                CC: darin at apple.com, sam at webkit.org, ossy at webkit.org,
                    noam at webkit.org, thorton at apple.com,
                    marcelo.lira at openbossa.org


http://trac.webkit.org/changeset/156472/trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp

324        m_useThreadedScrolling = parameters.store.getBoolValueForKey(WebPreferencesKey::threadedScrollingEnabledKey()); 
325        m_page->settings().setScrollingCoordinatorEnabled(m_useThreadedScrolling);

You can easily reproduce it on WebKitNix's MiniBrowser in desktop mode where scrolling doesn't work at all.
I think the lines above should be in #if ENABLE(THREADED_SCROLLING) guard.

USE(COORDINATED_GRAPHICS) platforms set ScrollingCoordinatorEnabled in WebPage::setUseFixedLayout(bool fixed) depends on mobile or desktop mode.

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