Issue 598533004: Enable a11y audits and fix issues for downloads page.

Issue 598533004: Enable a11y audits and fix issues for downloads page. (Closed)

Created:
6 years, 3 months ago by hcarmona

Modified:
6 years, 3 months ago

Reviewers:
dmazzoni, Dan Beam

CC:
chromium-reviews, asanka, benjhayden+dwatch_chromium.org, arv+watch_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Project:
chromium

Visibility:
Public.

Enable a11y audits and fix issues for downloads page. BUG=265583 Committed: https://crrev.com/3e08a1707f2b2406664e325772da6968f23ca940 Cr-Commit-Position: refs/heads/master@{#296513}

Created: 6 years, 3 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+8 lines, -0 lines)			Patch
	M	chrome/browser/resources/downloads/downloads.js	View		2 chunks	+2 lines, -0 lines	0 comments	Download
	M	chrome/browser/ui/webui/downloads_ui_browsertest.js	View		1 chunk	+6 lines, -0 lines	0 comments	Download

Total messages: 8 (2 generated)

hcarmona

Enabled a11y audit and it only found an issue with the download icons. Added empty ...

6 years, 3 months ago (2014-09-24 17:08:10 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/598533004/1

6 years, 3 months ago (2014-09-24 20:43:29 UTC) #6

commit-bot: I haz the power

Committed patchset #1 (id:1) as 980536cb88d7e68df5834de363445c5cf39e03cd

6 years, 3 months ago (2014-09-24 20:56:50 UTC) #7

commit-bot: I haz the power

6 years, 3 months ago (2014-09-24 20:57:22 UTC) #8

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/3e08a1707f2b2406664e325772da6968f23ca940
Cr-Commit-Position: refs/heads/master@{#296513}