MD Downloads: fix some a11y issues

MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/382d94ea2fa27cbd874b679a955f63ca804279df
Cr-Commit-Position: refs/heads/master@{#437603}

[Dan Beam, 2016-12-08 04:29:28]

Description was changed from

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles

R=calamity@chromium.org
BUG=none
==========

to

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-12-08 04:30:27]

The CQ bit was checked by dbeam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-08 04:30:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-08 05:25:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-08 05:25:03]

Dry run: This issue passed the CQ dry run.

[calamity, 2016-12-09 01:31:42]

lgtm w/ question.

https://codereview.chromium.org/2555693006/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/item.html (right):

https://codereview.chromium.org/2555693006/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/item.html:33: color: rgb(103, 112, 115);
Is this a remnant from the old blue-grey color days? Should we ask Alan if we
can just make this grey?

[Dan Beam, 2016-12-09 02:03:33]

Description was changed from

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-12-09 02:03:54]

https://codereview.chromium.org/2555693006/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/item.html (right):

https://codereview.chromium.org/2555693006/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/item.html:33: color: rgb(103, 112, 115);
On 2016/12/09 01:31:42, calamity wrote:
> Is this a remnant from the old blue-grey color days? Should we ask Alan if we
> can just make this grey?

talked with bettes@, says it's cool to use grey 700 here and in settings

[Dan Beam, 2016-12-09 02:03:58]

The CQ bit was checked by dbeam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 02:04:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 03:26:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-09 03:26:34]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-12-09 18:53:33]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-12-09 18:53:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
calamity@chromium.org
Link to the patchset: https://codereview.chromium.org/2555693006/#ps20001
(title: "more nits")

[commit-bot: I haz the power, 2016-12-09 18:54:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 19:11:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481309613157640,
"parent_rev": "f7ca9b6a095a061dea1b02e8775745b9dd158093", "commit_rev":
"d33ee282a752bdc6ab74daa2f52cf1505a20069c"}

[commit-bot: I haz the power, 2016-12-09 19:12:12]

Description was changed from

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2555693006
==========

[commit-bot: I haz the power, 2016-12-09 19:12:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-12 14:36:43]

Description was changed from

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2555693006
==========

to

==========
MD Downloads: fix some a11y issues

* up contrast to 4.5:1 for small text
* fix some aria roles
* update header colors (and make settings match while I'm at it)

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/382d94ea2fa27cbd874b679a955f63ca804279df
Cr-Commit-Position: refs/heads/master@{#437603}
==========

[commit-bot: I haz the power, 2016-12-12 14:36:44]

Patchset 2 (id:??) landed as
https://crrev.com/382d94ea2fa27cbd874b679a955f63ca804279df
Cr-Commit-Position: refs/heads/master@{#437603}