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

Issue 762430 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug
Team-Security-UX

Blocking:
issue 751949
issue 762333



Sign in to add a comment

Expand NavigationThrottleResult to include net error and optional error page HTML

Project Member Reported by lgar...@chromium.org, Sep 6 2017

Issue description

Components: UI>Browser>Interstitials
Labels: M-63
Status: Fixed (was: Started)
I have a CL out for a documentation typo, but this is implemented in https://chromium.googlesource.com/chromium/src/+/75d2c9f905a4e994d6f41c5104a24f7a0e4745ac
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 27 2017

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

commit 123b6a15aabf3f38abc1c633383924d652bb433d
Author: Lucas Garron <lgarron@chromium.org>
Date: Wed Sep 27 23:41:00 2017

Update NavigationThrottle::CancelDeferredNavigation() comment to specifically refer to the ThrottleCheckResult's action.

Bug:  762430 
Change-Id: Ieaea62732d604ca56fb3cdb97e96a4ac98e76f15
Reviewed-on: https://chromium-review.googlesource.com/686102
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Commit-Queue: Lucas Garron <lgarron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504803}
[modify] https://crrev.com/123b6a15aabf3f38abc1c633383924d652bb433d/content/public/browser/navigation_throttle.h

Sign in to add a comment