New issue
Advanced search Search tips

Issue 736304 link

Starred by 1 user
Status: Fixed
Owner:
jochen@chromium.org
Closed: Jun 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

[Missing Tests]: <a download=""> value is ignored

Project Member Reported by krajshree@chromium.org, Jun 23 2017 
Automated tests for the below commit have been missing.Would it be possible to add test coverage to avoid regressions in future?

CL: 
https://chromium.googlesource.com/chromium/src.git/+/e09ad54e01d9a94d9bc0480ed7f696f2ca188143

Ref Bug: 
https://bugs.chromium.org/p/chromium/issues/detail?id=730050

Thank you!
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 29 2017 

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

commit 9cb51a30376e75d2de0d20db35dc346d0fcf92f4
Author: Jochen Eisinger <jochen@chromium.org>
Date: Thu Jun 29 20:05:44 2017

Test that suggested filenames for data: URLs work

BUG= 730050 ,  736304 
R=dtrainor@chromium.org

Change-Id: I2d0e02adee090787858aea446eafaf9efa22256d
Reviewed-on: https://chromium-review.googlesource.com/549896
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#483459}
[modify] https://crrev.com/9cb51a30376e75d2de0d20db35dc346d0fcf92f4/content/browser/download/download_browsertest.cc

Comment 2 by jochen@chromium.org, Jun 30 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment