[CB-7947] pauseTimers() called too aggressively on Android

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: cordova-android
• Assignee: Andrew Grieve
• Reporter: Andrew Grieve
• Created: 05/Nov/14 16:02
• Updated: 23/Aug/18 07:08
• Resolved: 27/Jan/15 02:43

Description

ML discussion: http://markmail.org/thread/dhs7lnnrhq6z24vb

Couple of issues:
1. pauseTimers(), although an instance method, actually applies to all webviews
2. pauseTimers() is always being called during startActivityForResult(), even though there's no reason to do this.

Proposed changes:

• Pause timers during a startActivityForResult only when KeepRunning preference is false (this should happen anyways, since the activity's onPause is fired in this case)
• 4.0.x branch: Create a setGloballyPauseTimers(boolean) on CordovaWebview interface in 4.0.x that is documented to have the same semantics as WebView.pause/resume timers.
• 4.0.x branch: Move the pausing & resuming into CordovaActivity, rather than passing it as a boolean to the onPause() function

Issue Links

• relates to: CB-7197 Cordova doesn't always initialize when webview timers start in the paused state