Bug 126547

Summary:

Hook up the RemoteScrollingCoordinator

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

New Bugs

Assignee:

Simon Fraser (smfr) <simon.fraser>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, bdakin, philn, sam, simon.fraser, thorton, xan.lopez

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	thorton: review+

Description Simon Fraser (smfr) 2014-01-06 16:17:07 PST

Hook up the RemoteScrollingCoordinator

Comment 1 Simon Fraser (smfr) 2014-01-06 16:23:48 PST

Created attachment 220468 [details]
Patch

Comment 2 Tim Horton 2014-01-06 16:27:19 PST

Comment on attachment 220468 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=220468&action=review

> Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:242
> +        encoder << Vector<ScrollingNodeID>();

why?

> Source/WebKit2/UIProcess/WebPageProxy.cpp:1245
> +    if (m_scrollingCoordinatorProxy && m_scrollingCoordinatorProxy->handleWheelEvent(platform(event)))

should this be in an #if too?

Comment 3 Simon Fraser (smfr) 2014-01-06 16:53:48 PST

https://trac.webkit.org/r161387