[ 
https://issues.apache.org/jira/browse/CB-10498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15132684#comment-15132684
 ] 

ASF GitHub Bot commented on CB-10498:
-------------------------------------

Github user riknoll commented on a diff in the pull request:

    https://github.com/apache/cordova-android/pull/257#discussion_r51911630
  
    --- Diff: cordova-js-src/platform.js ---
    @@ -58,6 +61,19 @@ module.exports = {
             bindButtonChannel('volumeup');
             bindButtonChannel('volumedown');
     
    +        // The resume event is not "sticky", but it is possible that the 
event
    +        // will contain the result of a plugin call. We need to ensure 
that the
    +        // plugin result is delivered even after the event is fired 
(CB-10498)
    +        var cordovaAddEventListener = document.addEventListener;
    --- End diff --
    
    @adamduren We already patch `addEventListener` in the common Cordova js 
code (hence the `cordovaAddEventListener` name). See 
https://github.com/apache/cordova-js/blob/master/src/cordova_b.js#L48
    
    Come to think of it, I should probably be doing the method in window as 
well. I'll update the PR. Thanks for taking a look!


> Resume event not guaranteed to be received with plugin result
> -------------------------------------------------------------
>
>                 Key: CB-10498
>                 URL: https://issues.apache.org/jira/browse/CB-10498
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android
>            Reporter: Richard B Knoll
>            Assignee: Richard B Knoll
>              Labels: android, triaged
>
> The save/restore API delivers results in the resume event. The issue is, it 
> is possible to "miss" the resume event if you don't add an event handler in 
> time.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to