CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players, create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode (--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332
Committed: https://crrev.com/4c44bddab18803749299a912907601acff719911
Cr-Commit-Position: refs/heads/master@{#356006}

[loyso (OOO), 2015-10-19 04:11:26]

loyso@chromium.org changed reviewers:
+ chrishtr@chromium.org, vollick@chromium.org

[Ian Vollick, 2015-10-20 05:16:36]

On 2015/10/19 04:31:38, loyso wrote:

This seems straightforward. lgtm.

[loyso (OOO), 2015-10-20 05:43:57]

Description was changed from

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=394777
==========

to

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332
==========

[loyso (OOO), 2015-10-20 05:44:04]

The CQ bit was checked by loyso@chromium.org

[loyso (OOO), 2015-10-20 05:44:08]

The CQ bit was unchecked by loyso@chromium.org

[loyso (OOO), 2015-10-21 06:17:57]

chrishtr@, friendly ping!

[loyso (OOO), 2015-10-26 00:07:07]

loyso@chromium.org changed reviewers:
+ pdr@chromium.org

[loyso (OOO), 2015-10-26 00:07:24]

pdr@, could you take a look? Thanks!

[pdr., 2015-10-26 02:30:27]

lgtm

[loyso (OOO), 2015-10-26 02:37:43]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2015-10-26 02:37:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1410873003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1410873003/1

[commit-bot: I haz the power, 2015-10-26 03:29:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-26 03:30:13]

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

[loyso (OOO), 2015-10-26 03:43:03]

Description was changed from

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332

Committed: https://crrev.com/4c44bddab18803749299a912907601acff719911
Cr-Commit-Position: refs/heads/master@{#356006}
==========

to

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332

Committed: https://crrev.com/4c44bddab18803749299a912907601acff719911
Cr-Commit-Position: refs/heads/master@{#356006}

Fixes after it was reverted:
crrev.com/1410873003 Fix isThreadedAnimationEnabled mode for layout tests
crrev.com/1413003002 Fix AnimationHost mode integration with Property Trees.
==========

[loyso (OOO), 2015-10-26 03:43:54]

Description was changed from

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332

Committed: https://crrev.com/4c44bddab18803749299a912907601acff719911
Cr-Commit-Position: refs/heads/master@{#356006}

Fixes after it was reverted:
crrev.com/1410873003 Fix isThreadedAnimationEnabled mode for layout tests
crrev.com/1413003002 Fix AnimationHost mode integration with Property Trees.
==========

to

==========
CC Animations: Fix isThreadedAnimationEnabled mode for layout tests

Distinguish three modes for CC compositor animation timelines (CC AnimationHost)

1) AnimationHost disabled: No players, no timelines.

2) AnimationHost enabled in --enable-threaded-compositing mode: Create players,
create timelines.

3) AnimationHost enabled without --enable-threaded-compositing mode
(--disable-threaded-animation mode): Create players, do not create timelines.

ProgrammaticScrollAnimator already supports the case when timeline is nullptr.

BUG=545332

Committed: https://crrev.com/4c44bddab18803749299a912907601acff719911
Cr-Commit-Position: refs/heads/master@{#356006}
==========