New issue
Advanced search Search tips

Issue 890533 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Oct 10
Components:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 2
Type: Bug



Sign in to add a comment

Navigation load error URLs are not transmitted via NavigationEvent interface.

Project Member Reported by kmarshall@chromium.org, Sep 29

Issue description

When the browser hits an error when loading a webpage (e.g. for a site with an unresolvable domain), the NavigationEvent interface reports the URL as chrome-error://chromewebdata/, instead of the URL that caused the error.
 
Labels: M-71
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 1

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

commit 06d9324b620f04fe0d214d247dbf12a4ef495135
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Mon Oct 01 23:55:53 2018

[fuchsia] Use content::NavigationEntry data when building FIDL events.

content::NavigationEntry contains more useful data about the
page load state compared to |validated_url|, which obscures the URLs
that caused page load errors.

Add an |is_error| field to indicate if an error occurred during
navigation.

BUG= 890533 

Change-Id: If7407cec236ddd5b1474f52ac2538b3a3657a366
Reviewed-on: https://chromium-review.googlesource.com/1252637
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595647}
[modify] https://crrev.com/06d9324b620f04fe0d214d247dbf12a4ef495135/webrunner/browser/context_impl_browsertest.cc
[modify] https://crrev.com/06d9324b620f04fe0d214d247dbf12a4ef495135/webrunner/browser/frame_impl.cc
[modify] https://crrev.com/06d9324b620f04fe0d214d247dbf12a4ef495135/webrunner/fidl/navigation_controller.fidl
[modify] https://crrev.com/06d9324b620f04fe0d214d247dbf12a4ef495135/webrunner/fidl/navigation_event_observer.fidl

Status: Verified (was: Assigned)

Sign in to add a comment