android: move overscroll effects to ui/

android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfill all the dependencies. The code here mainly depends on cc.

BUG=507792
NOTRY=True

Committed: https://crrev.com/a964cb746a2b7031188199d6cb7d5459868d9c7b
Cr-Commit-Position: refs/heads/master@{#355829}

[no sievers, 2015-10-23 00:11:14]

Description was changed from

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfull all the dependencies.
(It also really doesn't need to be Android-specific, but mainly
depends on cc.)

BUG=507792
==========

to

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfull all the dependencies.
(It also really doesn't need to be Android-specific, but mainly
depends on cc.)

BUG=507792
==========

[no sievers, 2015-10-23 00:11:14]

sievers@chromium.org changed reviewers:
+ aurimas@chromium.org

[no sievers, 2015-10-23 00:15:14]

sievers@chromium.org changed reviewers:
+ danakj@chromium.org

[no sievers, 2015-10-23 00:15:23]

+Dana for ui/android/DEPS

[aurimas (slooooooooow), 2015-10-23 00:16:06]

aurimas@chromium.org changed reviewers:
- danakj@chromium.org

[aurimas (slooooooooow), 2015-10-23 00:16:06]

LGTM. The move seems reasonable especially given there are no other dependencies

[no sievers, 2015-10-23 00:17:26]

Description was changed from

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfull all the dependencies.
(It also really doesn't need to be Android-specific, but mainly
depends on cc.)

BUG=507792
==========

to

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfull all the dependencies. The code here mainly depends on cc.

BUG=507792
==========

[no sievers, 2015-10-23 00:17:39]

Description was changed from

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfull all the dependencies. The code here mainly depends on cc.

BUG=507792
==========

to

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfill all the dependencies. The code here mainly depends on cc.

BUG=507792
==========

[danakj, 2015-10-23 00:52:36]

danakj@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2015-10-23 00:52:37]

LGTM

[no sievers, 2015-10-23 15:55:06]

The CQ bit was checked by sievers@chromium.org

[no sievers, 2015-10-23 15:55:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
aurimas@chromium.org, danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1419123002/#ps80001
(title: "missing file")

[commit-bot: I haz the power, 2015-10-23 15:55:37]

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

[commit-bot: I haz the power, 2015-10-23 16:03:09]

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

[commit-bot: I haz the power, 2015-10-23 16:03:10]

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...)

[no sievers, 2015-10-23 16:28:04]

sievers@chromium.org changed reviewers:
+ dtrainor@chromium.org

[David Trainor- moved to gerrit, 2015-10-23 16:33:21]

lgtm

[no sievers, 2015-10-23 16:48:46]

The CQ bit was checked by sievers@chromium.org

[no sievers, 2015-10-23 16:48:46]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
aurimas@chromium.org, dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/1419123002/#ps100001
(title: "fix exports for component build")

[commit-bot: I haz the power, 2015-10-23 16:49:20]

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

[commit-bot: I haz the power, 2015-10-23 17:54:26]

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

[commit-bot: I haz the power, 2015-10-23 17:54:27]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[no sievers, 2015-10-23 17:56:00]

Description was changed from

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfill all the dependencies. The code here mainly depends on cc.

BUG=507792
==========

to

==========
android: move overscroll effects to ui/

Instead of shoving this into content, move this down.
This separates this out nicer from the Aura+Android build which doesn't
fulfill all the dependencies. The code here mainly depends on cc.

BUG=507792
NOTRY=True
==========

[no sievers, 2015-10-23 17:56:05]

The CQ bit was checked by sievers@chromium.org

[commit-bot: I haz the power, 2015-10-23 17:56:38]

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

[commit-bot: I haz the power, 2015-10-23 18:06:46]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-10-23 18:07:47]

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

[jdduke (slow), 2015-10-23 18:12:16]

Yay :) I tried to minimize any explicit content/ dependencies when writing this
code, hopefully it wasn't super painful...

[no sievers, 2015-10-23 18:14:57]

On 2015/10/23 18:12:16, jdduke wrote:
> Yay :) I tried to minimize any explicit content/ dependencies when writing
this
> code, hopefully it wasn't super painful...

Ended up being simple, since it only depended on compositor for which we already
had the ui::WindowAndroidCompositor interface now.