[Chromium] Update forked ScrollAnimatorChromiumMac.mm/h

[Sailesh Agrawal]

[Chromium] Update forked ScrollAnimatorChromiumMac.mm/h

[Sailesh Agrawal]

Created attachment 97404 [details]
Patch

[James Robinson]

Comment on attachment 97404 [details]
Patch

OK.  What does the delta look like between ScrollAnimatorChromiumMac vs ScrollAnimatorMac currently?

[Sailesh Agrawal]

(In reply to comment #2)
> (From update of attachment 97404 [details])
> OK.  What does the delta look like between ScrollAnimatorChromiumMac vs ScrollAnimatorMac currently?

Do you mean after this patch? It's not much:
  - change compile time #if USE(WK_SCROLLBAR_PAINTER) to runtime if (isScrollbarOverlayAPIAvailable())
  - early out in ScrollAnimatorChromiumMac::scroll if smooth scrolling is disabled
  - rename ScrollbarThemeMac to ScrollbarThemeChromiumMac

[Sailesh Agrawal]

Layout try bots look good:
http://build.chromium.org/p/tryserver.chromium/builders/mac_layout_rel/builds/208
http://build.chromium.org/p/tryserver.chromium/builders/linux_layout_rel/builds/273

(windows one is still pending).

[WebKit Review Bot]

Comment on attachment 97404 [details]
Patch

Clearing flags on attachment: 97404

Committed r89011: <http://trac.webkit.org/changeset/89011>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Sailesh Agrawal]

Re-opening since the original change got backed out.

[Sailesh Agrawal]

Created attachment 97473 [details]
Patch

[Sailesh Agrawal]

Merged latest changes from this revision:
https://trac.webkit.org/changeset/88982

Please take another look.

[Mihai Parparita]

Comment on attachment 97473 [details]
Patch

Rubber-stamping since James is out.

[WebKit Review Bot]

Comment on attachment 97473 [details]
Patch

Clearing flags on attachment: 97473

Committed r89066: <http://trac.webkit.org/changeset/89066>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.