CC Animation: Erase old animation system in Android Compositor.

CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041
Committed: https://crrev.com/e7c252c14c9afe6ef128f2c0c7bd75e4a1c14e73
Cr-Commit-Position: refs/heads/master@{#380587}

[loyso (OOO), 2016-02-29 02:33:15]

Description was changed from

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

BUG=575041
==========

to

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

BUG=575041
==========

[loyso (OOO), 2016-02-29 02:33:15]

loyso@chromium.org changed reviewers:
+ dtrainor@chromium.org, piman@chromium.org

[loyso (OOO), 2016-02-29 02:38:32]

Description was changed from

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

BUG=575041
==========

to

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041
==========

[loyso (OOO), 2016-02-29 03:39:36]

loyso@chromium.org changed reviewers:
- dtrainor@chromium.org, piman@chromium.org

[loyso (OOO), 2016-02-29 03:40:52]

loyso@chromium.org changed reviewers:
+ dtrainor@chromium.org, kinuko@chromium.org

[loyso (OOO), 2016-02-29 06:14:37]

https://codereview.chromium.org/1747783002/diff/80001/chrome/browser/android/...
File chrome/browser/android/DEPS (right):

https://codereview.chromium.org/1747783002/diff/80001/chrome/browser/android/...
chrome/browser/android/DEPS:10: "+cc/layers/layer_settings.h",
This is temp and will be deleted in next CL along with layer_settings.h/cc

[kinuko, 2016-02-29 09:02:38]

content/ changes lgtm

[loyso (OOO), 2016-03-01 01:39:04]

loyso@chromium.org changed reviewers:
+ bauerb@chromium.org, miguelg@chromium.org

[loyso (OOO), 2016-03-01 01:40:18]

ui/android, chrome/browser/android review needed! PTAL, this is a trivial
deletion.

[Bernhard Bauer, 2016-03-01 10:33:15]

RSLGTM

[Bernhard Bauer, 2016-03-01 10:33:49]

On 2016/03/01 10:33:15, Bernhard Bauer wrote:
> RSLGTM

(Which Rietveld doesn't recognize, I guess? Rubberstamp LGTM then.)

[newt (away), 2016-03-02 02:23:41]

newt@chromium.org changed reviewers:
+ newt@chromium.org, sievers@chromium.org

[newt (away), 2016-03-02 02:23:42]

sievers: Could you take a look at ui/android? Looks like you moved these files
into ui recently.

[loyso (OOO), 2016-03-03 02:22:57]

PTAL, anyone!

[loyso (OOO), 2016-03-03 02:24:04]

loyso@chromium.org changed reviewers:
- newt@chromium.org, sievers@chromium.org

[loyso (OOO), 2016-03-03 23:01:35]

Friendly ping! This is trivial deletion which blocks ongoing migration.

[no sievers, 2016-03-07 19:16:52]

On 2016/03/02 02:23:42, newt wrote:
> sievers: Could you take a look at ui/android? Looks like you moved these files
> into ui recently.

lgtm!

[loyso (OOO), 2016-03-07 23:55:37]

On 2016/03/07 19:16:52, sievers wrote:
> On 2016/03/02 02:23:42, newt wrote:
> > sievers: Could you take a look at ui/android? Looks like you moved these
files
> > into ui recently.
> lgtm!

Hmm. sievers@ isn't an OWNER for ui/android.
ui/android review still needed! PTAL.

[David Trainor- moved to gerrit, 2016-03-08 18:55:11]

ui/android lgtm

[loyso (OOO), 2016-03-08 22:42:02]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2016-03-08 22:42:31]

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

[commit-bot: I haz the power, 2016-03-08 22:55:35]

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

[commit-bot: I haz the power, 2016-03-08 22:55:36]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[loyso (OOO), 2016-03-08 22:59:32]

** Presubmit ERRORS **
Missing LGTM from OWNERS of dependencies added to DEPS:
    '+cc/layers/layer_settings.h',

Hmm. This is strange.

[loyso (OOO), 2016-03-08 23:02:14]

On 2016/03/08 22:59:32, loyso wrote:
> Missing LGTM from OWNERS of dependencies added to DEPS:
>     '+cc/layers/layer_settings.h',

dtrainor@, review chrome/browser/android/DEPS please? This is a temp change. We
will erase layer_settings.h in next CL.

[David Trainor- moved to gerrit, 2016-03-10 00:05:31]

chrome/browser/android/deps lgtm

[loyso (OOO), 2016-03-10 00:07:45]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2016-03-10 00:08:38]

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

[commit-bot: I haz the power, 2016-03-10 00:19:28]

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

[commit-bot: I haz the power, 2016-03-10 00:19:30]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[loyso (OOO), 2016-03-10 00:28:40]

Still
** Presubmit ERRORS **
Missing LGTM from OWNERS of dependencies added to DEPS:
    '+cc/layers/layer_settings.h',

[loyso (OOO), 2016-03-10 00:34:35]

Filed a bug for Infra:
https://bugs.chromium.org/p/chromium/issues/detail?id=593533

[loyso (OOO), 2016-03-10 00:35:37]

On 2016/03/10 00:05:31, David Trainor wrote:
> chrome/browser/android/deps lgtm

chrome/browser/android/DEPS maybe? :)

[Ian Vollick, 2016-03-11 02:34:01]

On 2016/03/10 00:35:37, loyso wrote:
> On 2016/03/10 00:05:31, David Trainor wrote:
> > chrome/browser/android/deps lgtm
> 
> chrome/browser/android/DEPS maybe? :)

lgtm.

[loyso (OOO), 2016-03-11 02:51:35]

The CQ bit was checked by loyso@chromium.org

[loyso (OOO), 2016-03-11 02:51:35]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org,
bauerb@chromium.org, dtrainor@chromium.org, sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1747783002/#ps100001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-03-11 02:51:53]

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

[commit-bot: I haz the power, 2016-03-11 02:51:55]

This CL has an open dependency (Issue 1782433002 Patch 60001). Please resolve
the dependency and try again.

[loyso (OOO), 2016-03-11 08:08:11]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2016-03-11 08:08:33]

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

[commit-bot: I haz the power, 2016-03-11 09:40:35]

Description was changed from

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041
==========

to

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041
==========

[commit-bot: I haz the power, 2016-03-11 09:40:37]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-03-11 09:42:06]

Description was changed from

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041
==========

to

==========
CC Animation: Erase old animation system in Android Compositor.

Basically, specialize all the code as if use_compositor_animation_timelines is
always true.

Next CL: Erase cc::LayerSettings everywhere.

BUG=575041

Committed: https://crrev.com/e7c252c14c9afe6ef128f2c0c7bd75e4a1c14e73
Cr-Commit-Position: refs/heads/master@{#380587}
==========

[commit-bot: I haz the power, 2016-03-11 09:42:07]

Patchset 6 (id:??) landed as
https://crrev.com/e7c252c14c9afe6ef128f2c0c7bd75e4a1c14e73
Cr-Commit-Position: refs/heads/master@{#380587}