New issue
Advanced search Search tips

Issue 795276 link

Starred by 0 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug



Sign in to add a comment

Specific webpage from fcc.gov is not loading.

Project Member Reported by srikanthg@chromium.org, Dec 15 2017

Issue description

App Version: 64.0.3282.31 beta
iOS Version: 11.2, 11.1
Device: iPhone7, iPhoneX, iPad
URL: https://www.fcc.gov/ecfs/search/proceedings?q=name:((17-108))

Steps to reproduce:
  1. Launch Google Chrome
  2. Navigate to the above URL (or go link: https://goo.gl/8usYgv)

Observed results: on iPad: Complete blank page is displayed
On iPhone: Page is loaded partially without any text content. 
Page is not loaded even with changing Safari User Agent ON.

Expected results: Page should be loaded completely.

Note: Working fine in Safari, Firefox and WKWebView app

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 (Tested on M65 canary, Nexus5)
Bug reproducible on Safari/Firefox: Firefox: NO, Safari: NO
Bug reproducible on current stable build (App Version, iOS Version): M63 YES
Bug reproducible on the current beta channel build (App Version, iOS Version): M64 YES

Link to video/image: https://docs.google.com/drawings/d/1Gn_im1Eztjo_aLEvG2R56xtRyFuLK6M5j8J8fiJTp9Y/preview 
 
Cc: -eugene...@chromium.org
Labels: -Restrict-View-Google -Pri-2 ReleaseBlock-Stable M-64 Pri-1
Owner: eugene...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/32544576b5bb1b3dac95496671c5f69228dc3184

commit 32544576b5bb1b3dac95496671c5f69228dc3184
Author: Eugene But <eugenebut@google.com>
Date: Mon Dec 18 21:52:57 2017

Gracefully handle exceptions in console JS API.

.join(' ') call can throw an exception, which should not leave overriden
console API.

Bug:  795276 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I50b7a9fe73eed4c838794818267ee5f387e4ac89
Reviewed-on: https://chromium-review.googlesource.com/830907
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524814}
[modify] https://crrev.com/32544576b5bb1b3dac95496671c5f69228dc3184/ios/web/web_state/js/resources/console.js

Labels: Merge-Request-64
Status: Fixed (was: Started)
Project Member

Comment 4 by sheriffbot@chromium.org, Dec 18 2017

Labels: -Merge-Request-64 Hotlist-Merge-Review Merge-Review-64
This bug requires manual review: M64 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by cma...@chromium.org, Dec 21 2017

Has this been verified in canary?

Comment 6 by cma...@chromium.org, Dec 21 2017

Labels: -Hotlist-Merge-Review -Merge-Review-64 Merge-Approved-64
Merge approved upon verification of the fix in canary/dev
Status: Verified (was: Fixed)
Webpage loads fine on M65.0.3300.0 canary
Verified on iPhoneX 11.2.5, iPad Air 10.3.3
Project Member

Comment 8 by bugdroid1@chromium.org, Dec 22 2017

Labels: -merge-approved-64 merge-merged-3282
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2ae61e0b086da4c05484cd071b4da48ce239b166

commit 2ae61e0b086da4c05484cd071b4da48ce239b166
Author: Eugene But <eugenebut@google.com>
Date: Fri Dec 22 18:55:59 2017

Gracefully handle exceptions in console JS API.

.join(' ') call can throw an exception, which should not leave overriden
console API.

TBR=eugenebut@google.com

(cherry picked from commit 32544576b5bb1b3dac95496671c5f69228dc3184)

Bug:  795276 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I50b7a9fe73eed4c838794818267ee5f387e4ac89
Reviewed-on: https://chromium-review.googlesource.com/830907
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#524814}
Reviewed-on: https://chromium-review.googlesource.com/843053
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#343}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/2ae61e0b086da4c05484cd071b4da48ce239b166/ios/web/web_state/js/resources/console.js

Webpage loads fine on M64.0.3282.69 Beta
Verified on iPhone7 11.2.5, iPad Air 11.2.5 beta and iPad pro 10.3.3

Sign in to add a comment