[
https://issues.apache.org/jira/browse/CB-8383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15315334#comment-15315334
]
ASF GitHub Bot commented on CB-8383:
------------------------------------
Github user codecov-io commented on the issue:
https://github.com/apache/cordova-android/pull/288
## [Current coverage][cc-pull] is **33.29%**
> Merging [#288][cc-pull] into [master][cc-base-branch] will decrease
coverage by **2.99%**
```diff
@@ master #288 diff @@
=====================================
Files 11 11
Lines 926 916 -10
Methods 0 0
Branches 167 167
=====================================
- Hits 336 305 -31
- Misses 590 610 +20
- Partials 0 1 +1
```
> Powered by [Codecov](https://codecov.io?src=pr). Last updated by
[9af7f50...98d1460][cc-compare]
[cc-base-branch]:
https://codecov.io/gh/apache/cordova-android/branch/master?src=pr
[cc-compare]:
https://codecov.io/gh/apache/cordova-android/compare/9af7f506f03b468734406341ae98599dbd2af676...98d1460f808073dc94ad2cd1fa989156a7d8f91d
[cc-pull]: https://codecov.io/gh/apache/cordova-android/pull/288?src=pr
> pause event don't fire when 'KeepRunning' is false
> --------------------------------------------------
>
> Key: CB-8383
> URL: https://issues.apache.org/jira/browse/CB-8383
> Project: Apache Cordova
> Issue Type: Bug
> Components: Android
> Affects Versions: Master, 3.5.0, 4.0.0, 4.1.3
> Environment: Mac osx 10.9
> Reporter: hugefactory
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> Under Android, when a project use 'KeepRunning=false' in the config.xml file,
> the 'pause' event in javascript don't fire when application go to background.
> Next, 'pause' event fire when 'resume' is call and when the application
> resume.
> The problem is in 'CordovaWebView.java' in the 'handlePause' method :
> public void handlePause(boolean keepRunning) {
> LOG.d(TAG, "Handle the pause");
> // Send pause event to JavaScript
>
> this.loadUrl("javascript:try{cordova.fireDocumentEvent('pause');}catch(e){console.log('exception
> firing pause event from native');};");
> // Forward to plugins
> if (this.pluginManager != null) {
> this.pluginManager.onPause(keepRunning);
> }
> // If app doesn't want to run in background
> if (!keepRunning) {
> // Pause JavaScript timers (including setInterval)
> this.pauseTimers();
> }
> paused = true;
>
> }
> As you can see, 'pauseTimers()' is call immediatly when we choose
> 'KeepRunning' false.
> According to the Android documentation, 'pauseTimer()' stop all javascript
> activity....then the 'pause' javascript callback is not execute (it's execute
> after, when the 'resumeTimers()' is call and the application resume.
> I suggest to delay the 'pauseTimers()' for 1 or 2 sec. Then the javascript
> 'pause' can execute before the application is really stop :
> //OLD
> //this.pauseTimers();
> //NEW
> final Handler handler = new Handler();
> handler.postDelayed(new Runnable() {
> @Override
> public void run() {
> LOG.d(TAG, "Handle the pauseTimers");
> pauseTimers();
> }
> }, 1000);
> }
> Thanks.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
