New issue
Advanced search Search tips

Issue 796657 link

Starred by 1 user
Status: Fixed
Owner:
dougt@chromium.org
Closed: Dec 2017
Cc:
pastarmovj@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Add tests to ensure that Blacklist Policy correctly blocks redirects

Project Member Reported by dougt@chromium.org, Dec 20 2017 
In https://chromium-review.googlesource.com/c/chromium/src/+/745326, it was discovered that we do not have tests that ensure that the blacklist policy is correctly applied during a redirect when the network service is enabled.

We should add tests.
Project Member

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

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

commit bbcf5e2f58b3df00d32f46db569c01d5fb01ed8c
Author: Doug Turner <dougt@chromium.org>
Date: Fri Dec 22 21:08:38 2017

Add tests to ensure that Blacklist Policy correctly blocks redirects

Bug:  796657 
Change-Id: I49d2cd5fe8112fda7ec6ae6ae472ee13510d9394
Reviewed-on: https://chromium-review.googlesource.com/837730
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Doug Turner <dougt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526061}
[modify] https://crrev.com/bbcf5e2f58b3df00d32f46db569c01d5fb01ed8c/chrome/browser/policy/policy_browsertest.cc
[add] https://crrev.com/bbcf5e2f58b3df00d32f46db569c01d5fb01ed8c/chrome/test/data/policy/blacklist-redirect.html

Comment 2 by dougt@chromium.org, Dec 23 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment