Issue metadata
Sign in to add a comment
|
Omnibox freezes on canceling navigation to a certain URL, while the page is in loading state |
||||||||||||||||||||||
Issue descriptionApp Version: 72.0.3626.53 Beta iOS Version: 11.4.1, 12.0.1. 12.1, 12.1.2 Devices: iPhone 6 plus, iPhone X, iPhone 8, iPad Air, iPad Pro Prerequisite: Launch the app in landscape mode for iPhones Steps to reproduce: Launch iOS Chrome Navigate to any webpage Tap on ‘X’ button and cancel the navigation immediately Tap on Omnibox Observed results: Omnibox freezes on canceling a searched URL while the page is still loading Expected results: NTP page should be displayed on canceling a searched URL while the page is still not loaded 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 Safari/Firefox: Firefox: NA, Safari: NA Bug reproducible on the current stable build :M71, No Bug reproducible on the current beta channel build : M72, Yes Type-bug-regression? Yes Bad build: 72.0.3610.0 beta Revision no. 5bb637264a2f Good Build: 72.0.3609.3 beta Revision no. 7e51571a56ab Link to Image/Video: M72 https://drive.google.com/open?id=1PDgS2_LysCIujiK8Uc2f3N3aC8vZNdTx M71 https://drive.google.com/open?id=1JVDMB3zbrcmhQt69Ej9ZFC48zMmfYA-E
,
Jan 9
Is this bug reproducible with disabled browser-container-contains-ntp flag?
,
Jan 10
@eugene Unable to reproduce this issue after disabling the flag #browser-container-contains-ntp
,
Jan 10
Justin, can you take a look while Stepan is out.
,
Jan 11
It looks like this was fixed by an unrelated (sort of bug) crbug.com/917358 ghuphran@ can you confirm this doesn't happen in M73? If so, I think once the fix for 917358 is cherry picked to M72 this will be fixed as well.
,
Jan 11
[Auto-generated comment by a script] We noticed that this issue is targeted for M-72; it appears the fix may have landed after branch point, meaning a merge might be required. The owner of this bug should confirm if a merge is required here. If so, add Merge-Request-72 label and indicate which commits/CLs are to be merged. Otherwise, remove Merge-TBD label. Thanks.
,
Jan 11
@Justin I have tested this issue in the latest M73 canary, and the Omnibox freeze is not observed(refer Video 1). Build Version: 73.0.3667.0 Canary Devices: iPhoneX(12.0.1), iPhone8plus(11.4.1), iPad Air(12.1), iPad 6th Gen(11.4.1) Video 1: https://drive.google.com/file/d/1-_KUpPH29KvBLm9AwGlq6gkppJIPk7oF/view
,
Jan 11
I already requested a merge on 917358. Adding merge-request-72 here, but it's the same CL.
,
Jan 11
This bug requires manual review: Less than 14 days to go before AppStore submit on M72 Please contact the milestone owner if you have questions. Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 11
917358 was merged, so this no longer requires a merge. ghuphran@ can you verify again on the next beta to confirm it was indeed the same fix.
,
Jan 17
(6 days ago)
@Justin I have tested this issue in the latest M72 beta, and the Omnibox freeze is not observed Build Version: 72.0.3626.59 Beta Devices: iPhoneX(12.0.1), iPhone8plus(11.4.1), iPad Air(12.1), iPad 6th Gen(11.4.1) |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by linds...@chromium.org
, Jan 9Labels: ReleaseBlock-Stable M-72
Owner: stkhapugin@chromium.org
Status: Assigned (was: Untriaged)