rcorin opened a new issue, #1355:
URL: https://github.com/apache/cordova-ios/issues/1355

   # Bug Report
   
   ## Problem
   I'm investigating a situation when the app comes to foreground from 
background and the webview content process has been killed by the OS.
   In shouldReloadWebView , CDVWebViewEngine.m:283,  there is this code:
   
       BOOL title_is_nil = (title == nil);
       BOOL location_is_blank = [[location absoluteString] 
isEqualToString:@"about:blank"];
   
       BOOL reload = (title_is_nil || location_is_blank);
   
   however location is nil, hence is false and also title_is_nil is false, 
resulting in reload being false.
   
   ### What is expected to happen?
   
   If the location is nil, reload should be true, so the condition should be:
   
    BOOL location_is_blank = location == nil  || [[location absoluteString] 
isEqualToString:@"about:blank"];
   
   
   ### What does actually happen?
   
   the app gets stuck in a white screen because the reload isn't issued, due to 
shouldReloadWebView being false.
   
   ## Information
   <!-- Include all relevant information that might help understand and 
reproduce the problem -->
   
   To reproduce the issue, i had to actually run the app in simulator, kill the 
com.apple.WebKit.WebContent process with the app in background, and then 
comment out the other webview reloads until reaching the point of 
shouldReloadWebView.
   
   ### Command or Code
   <!-- What command or code is needed to reproduce the problem? -->
   
   See above info.
   
   ### Environment, Platform, Device
   <!-- In what environment, on what platform or on which device are you 
experiencing the issue? -->
   
   iOS simulator
   
   ### Version information
   <!-- 
   What are relevant versions you are using?
   For example:
   Cordova: Cordova CLI, Cordova Platforms, Cordova Plugins 
   Other Frameworks: Ionic Framework and CLI version
   Operating System, Android Studio, Xcode etc.
   -->
   
   Xcode 14.3, iOS 12, Cordoba 10.0.0.
   
   ## Checklist
   <!-- Please check the boxes by putting an x in the [ ] like so: [x] -->
   
   - [X ] I searched for existing GitHub issues
   - [ X] I updated all Cordova tooling to most recent version
   - [ X] I included all the necessary information above
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to