cc: Move OnScrollOffsetAnimated to scroll tree

cc: Move OnScrollOffsetAnimated to scroll tree

BUG=638297
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/0604b910a5bb2297cd6a4fc9c2557b1a0be48a4d
Cr-Commit-Position: refs/heads/master@{#418688}

[jaydasika, 2016-09-13 18:22:51]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org, weiliangc@chromium.org

[jaydasika, 2016-09-13 18:22:52]

PTAL

[jaydasika, 2016-09-13 18:23:03]

The CQ bit was checked by jaydasika@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 18:23:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-13 19:11:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 19:11:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[weiliangc, 2016-09-13 23:21:44]

LGTM.

Probably need someone else for the blink part?

[jaydasika, 2016-09-13 23:23:00]

jaydasika@chromium.org changed reviewers:
+ vollick@chromium.org

[jaydasika, 2016-09-13 23:23:00]

vollick : Can you review changes to platform/graphics ?

[Ian Vollick, 2016-09-14 17:10:57]

On 2016/09/13 23:23:00, jaydasika wrote:
> vollick : Can you review changes to platform/graphics ?

It's pretty clunky. CompositorMutableState shouldn't really own layers anymore
since that's not what you animate.

If you think that's too much to fix in this patch, I'd be happy doing that in a
follow up.

lgtm.

[jaydasika, 2016-09-14 17:12:40]

On 2016/09/14 17:10:57, Ian Vollick wrote:
> On 2016/09/13 23:23:00, jaydasika wrote:
> > vollick : Can you review changes to platform/graphics ?
> 
> It's pretty clunky. CompositorMutableState shouldn't really own layers anymore
> since that's not what you animate.
> 
> If you think that's too much to fix in this patch, I'd be happy doing that in
a
> follow up.
> 
> lgtm.

I will do it in a separate patch.

[jaydasika, 2016-09-14 17:13:43]

The CQ bit was checked by jaydasika@chromium.org

[commit-bot: I haz the power, 2016-09-14 17:13:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-14 21:51:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-14 21:53:38]

Description was changed from

==========
cc: Move OnScrollOffsetAnimated to scroll tree

BUG=638297
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Move OnScrollOffsetAnimated to scroll tree

BUG=638297
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/0604b910a5bb2297cd6a4fc9c2557b1a0be48a4d
Cr-Commit-Position: refs/heads/master@{#418688}
==========

[commit-bot: I haz the power, 2016-09-14 21:53:39]

Patchset 1 (id:??) landed as
https://crrev.com/0604b910a5bb2297cd6a4fc9c2557b1a0be48a4d
Cr-Commit-Position: refs/heads/master@{#418688}