New issue
Advanced search Search tips

Issue 679721 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

App crashes when reading list entries are opened in poor network (GIN-2g, GIN-3g)

Project Member Reported by vbarig...@chromium.org, Jan 10 2017

Issue description

App 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
 
Cc: linds...@chromium.org
Owner: olivierrobin@chromium.org
Status: Assigned (was: Untriaged)
Labels: ReleaseBlock-Stable M-57
Project Member

Comment 4 by bugdroid1@chromium.org, 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

Status: Fixed (was: Assigned)
Cc: mard...@chromium.org
FYI, this is the pocket crash.
Status: Verified (was: Fixed)
Verified in 58.0.2998.0 canary, iPhone 6+ iOS 10.2, 

App does not crash.

Sign in to add a comment