App crashes when reading list entries are opened in poor network (GIN-2g, GIN-3g) |
||||||
Issue descriptionApp Version: 57.0.2977.0 canary iOS Version: 10.2, 9.3.5 Device: iPhone 7 plus URL: https://www.yahoo.com/sports/news/kris-bryant-tied-the-knot-and-his-cubs-teammates-showed-up-in-style-155718204.html Steps to reproduce: 1. Launch chrome in a good network connection. 2. Open any URL, Ex: https://www.yahoo.com/sports/news/kris-bryant-tied-the-knot-and-his-cubs-teammates-showed-up-in-style-155718204.html 3. Go to Chrome Menu --> Share --> Read Later (Wait for 10 to 15 seconds here) 4. Switch to GIN-2g, GIN-3g through device settings. 5. Go to Chrome Menu --> Reading list --> Tap on the reading list entry. Observed results: App crashes Note: No crash log generated. Expected results: App should not crash. Number of times you were able to reproduce: 5/5 Bug reproducible after clean install: Yes Bug reproducible after clearing cache and cookies: Yes Bug reproducible on Chrome Mobile on Android: No Bug reproducible on Safari/Firefox: Firefox: Not tested, Safari: Not tested Bug reproducible on current stable build (App Version, iOS Version): NA - New feature on M57 Bug reproducible on the current beta channel build (App Version, iOS Version): NA - New feature on M57 Link to video/image: https://drive.google.com/a/google.com/file/d/0Bz2uwV55gGwDZEhGWUpaandRQTA/view?usp=sharing Device Log: https://drive.google.com/a/google.com/file/d/0Bz2uwV55gGwDRmVHZ1IxUldCa3c/view?usp=sharing
,
Jan 10 2017
,
Jan 10 2017
,
Jan 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e5a88e66c2120ec14fe98ec23da580300064dbbf commit e5a88e66c2120ec14fe98ec23da580300064dbbf Author: olivierrobin <olivierrobin@chromium.org> Date: Wed Jan 11 10:29:41 2017 Remove call to stopLoading when loading Distilled version. Stop loading destroys the pending entry which leads to a crash in release. Remove the call to stop. BUG= 679721 Review-Url: https://codereview.chromium.org/2623583005 Cr-Commit-Position: refs/heads/master@{#442856} [modify] https://crrev.com/e5a88e66c2120ec14fe98ec23da580300064dbbf/ios/chrome/browser/reading_list/reading_list_web_state_observer.mm
,
Jan 11 2017
,
Jan 16 2017
FYI, this is the pocket crash.
,
Jan 31 2017
Verified in 58.0.2998.0 canary, iPhone 6+ iOS 10.2, App does not crash. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by linds...@chromium.org
, Jan 10 2017