New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 724666 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug

Blocking:
issue 722827



Sign in to add a comment

Chrome DCHECKs when trying to open link in new window

Project Member Reported by peterlaurens@chromium.org, May 19 2017

Issue description

Chrome Version: 60.0.3101.0 (Developer Build)
OS: iOS

Chrome DCHECKs and crashes when following the open-in-new-tab link at this page:

https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_link_target

A DCHECK fails:
[0519/151755.625362:FATAL:favicon_cache.cc(405)] Check failed: page_url.is_valid(). 

What steps will reproduce the problem?
(1) Visit https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_link_target
(2) Tap VisitW3Schools.com!
(3) Crash

What is the expected result?
Less crashing.

What happens instead?
Crashes.
 
Chromium-2017-05-19-151855.crash
78.0 KB Download

Comment 1 by pkl@chromium.org, May 22 2017

Cc: eugene...@chromium.org
Labels: -Pri-3 Pri-2
Owner: jif@chromium.org
Status: Assigned (was: Untriaged)
Does this happen in M58/M59 too?
Cc: gambard@chromium.org
Labels: -Pri-2 Pri-1
I think this has been fixed here:
https://codereview.chromium.org/2880723002/

Peter, could you please check if this is still repro.
This still repros for me building TOT today Mon 22 (hash c28017fd850d3035cdfc11d688e07331d1e577fe).

Comment 4 by gab@chromium.org, May 25 2017

Blocking: 722827
This is different from the issue I solved with https://codereview.chromium.org/2880723002 .
This happens when an empty URL is opened: the FaviconCache tries to update the favicon of a page with an invalid (empty) URL.

Comment 6 by jif@chromium.org, Jun 9 2017

Status: Started (was: Assigned)

Comment 7 by jif@chromium.org, Jun 12 2017

Status: WontFix (was: Started)
Seems to have been fixed as I can't reproduce.

Sign in to add a comment