[GitHub] [cordova-android] breautek commented on issue #1319: In low memory, using overview button causes ondeviceready but not onresume

[GitBox <gi...@apache.org>]

breautek commented on issue #1319:
URL: https://github.com/apache/cordova-android/issues/1319#issuecomment-912804228


   I'm guessing the cause of this behaviour (and why `onResume` works in the camera plugin) is due to this if statement:
   
   https://github.com/apache/cordova-android/blob/dc4e065f619ce98d5c38211f5efd759d05959bec/framework/src/org/apache/cordova/CordovaWebViewImpl.java#L501-L506
   
   It seems like it intentionally prevents the resume event if the app has been "resumed" without a pause event. This assumption is 6 years old... but I do understand your rationale and I think I generally agree that Cordova should fire resume whenever native does a resume. I wonder what would be the consequences of changing this... it definitely would be a breaking change.
   
   I'm making an assumption here, can you confirm that in your test case, that the `pause` event is also never fired when you go into the "overview" screen? If that's the case, I think the above code is the reasoning why `resume` event is never fired.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org