Remove OnSuspend/OnResume notifications on Android.

Remove OnSuspend/OnResume notifications on Android.

These are incorrectly mixing disparate concepts. Chrome code expects
OnSuspend/OnResume to mean the system is about to go to sleep, not
just that there is no more visible activity.

This prevents background tasks like background audio from working
properly since the network connections are force killed. We could
workaround this by improving when OnSuspend/OnResume is sent, but
it seems no one really cares, so just delete these.

In fact a couple pieces of code explicitly reject OnSupend/OnResume
from Android since they do not mean what is expected. An alternate
signal for app suspension should be sent if this is ever needed
again in the future.

BUG=644515
TEST=background audio playback works.

Committed: https://crrev.com/610004f26299d1eb31b18f7dce3e2d3740eb8c04
Cr-Commit-Position: refs/heads/master@{#417691}

[DaleCurtis, 2016-09-08 20:03:12]

dalecurtis@chromium.org changed reviewers:
+ nyquist@chromium.org

[DaleCurtis, 2016-09-08 20:03:16]

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

[commit-bot: I haz the power, 2016-09-08 20:03:46]

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

[DaleCurtis, 2016-09-08 21:03:59]

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

[commit-bot: I haz the power, 2016-09-08 21:04:25]

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-09 00:38:46]

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

[commit-bot: I haz the power, 2016-09-09 00:38:47]

Dry run: This issue passed the CQ dry run.

[DaleCurtis, 2016-09-09 20:07:46]

dalecurtis@chromium.org changed reviewers:
+ agrieve@chromium.org

[DaleCurtis, 2016-09-09 20:07:47]

nyquist seems OOO, +agrieve

[agrieve, 2016-09-09 20:20:07]

On 2016/09/09 20:07:47, DaleCurtis wrote:
> nyquist seems OOO, +agrieve

lgtm

[DaleCurtis, 2016-09-09 20:22:51]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-09-09 20:23:22]

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-09 20:27:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-09 20:30:03]

Description was changed from

==========
Remove OnSuspend/OnResume notifications on Android.

These are incorrectly mixing disparate concepts. Chrome code expects
OnSuspend/OnResume to mean the system is about to go to sleep, not
just that there is no more visible activity.

This prevents background tasks like background audio from working
properly since the network connections are force killed. We could
workaround this by improving when OnSuspend/OnResume is sent, but
it seems no one really cares, so just delete these.

In fact a couple pieces of code explicitly reject OnSupend/OnResume
from Android since they do not mean what is expected. An alternate
signal for app suspension should be sent if this is ever needed
again in the future.

BUG=644515
TEST=background audio playback works.
==========

to

==========
Remove OnSuspend/OnResume notifications on Android.

These are incorrectly mixing disparate concepts. Chrome code expects
OnSuspend/OnResume to mean the system is about to go to sleep, not
just that there is no more visible activity.

This prevents background tasks like background audio from working
properly since the network connections are force killed. We could
workaround this by improving when OnSuspend/OnResume is sent, but
it seems no one really cares, so just delete these.

In fact a couple pieces of code explicitly reject OnSupend/OnResume
from Android since they do not mean what is expected. An alternate
signal for app suspension should be sent if this is ever needed
again in the future.

BUG=644515
TEST=background audio playback works.

Committed: https://crrev.com/610004f26299d1eb31b18f7dce3e2d3740eb8c04
Cr-Commit-Position: refs/heads/master@{#417691}
==========

[commit-bot: I haz the power, 2016-09-09 20:30:04]

Patchset 2 (id:??) landed as
https://crrev.com/610004f26299d1eb31b18f7dce3e2d3740eb8c04
Cr-Commit-Position: refs/heads/master@{#417691}