android: Remove redundant code to suspend timers in the background

android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327, 379158
Committed: https://crrev.com/31abb5e88e0dc5df1788e75d5096d3a9d4ad1e0c
Cr-Commit-Position: refs/heads/master@{#415951}

[Sami, 2016-09-01 13:47:05]

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

[commit-bot: I haz the power, 2016-09-01 13:47:21]

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

[Sami, 2016-09-01 13:54:17]

Description was changed from

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327
==========

to

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327
==========

[Sami, 2016-09-01 13:54:17]

skyostil@chromium.org changed reviewers:
+ miguelg@chromium.org

[Sami, 2016-09-01 13:56:15]

Note: we still need to keep the ViewMsg_SetWebKitSharedTimersSuspended because
it's used to implement WebView's pauseTimers() API.

[Miguel Garcia, 2016-09-01 14:03:38]

lgtm

can you link the cl to the the original bug as well?

[Sami, 2016-09-01 14:07:52]

Description was changed from

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327
==========

to

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327,379158
==========

[Sami, 2016-09-01 14:07:58]

The CQ bit was unchecked by skyostil@chromium.org

[Sami, 2016-09-01 14:08:02]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2016-09-01 14:08:08]

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-01 14:32:35]

Description was changed from

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327,379158
==========

to

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327,379158
==========

[commit-bot: I haz the power, 2016-09-01 14:32:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-01 14:35:02]

Description was changed from

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327,379158
==========

to

==========
android: Remove redundant code to suspend timers in the background

This patch removes the Activity-side code for suspending renderer timers
after 5 minutes in the background. This code is redundant because the
Blink scheduler also implements this timer suspension in the renderer.

BUG=637327,379158

Committed: https://crrev.com/31abb5e88e0dc5df1788e75d5096d3a9d4ad1e0c
Cr-Commit-Position: refs/heads/master@{#415951}
==========

[commit-bot: I haz the power, 2016-09-01 14:35:03]

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