MD Site Settings: Add icons for a few categories.

MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/8e1b4e7ae7d21f64f85dad5d35235fd443277cae
Cr-Commit-Position: refs/heads/master@{#397588}

[Finnur, 2016-06-02 20:01:18]

Description was changed from

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 543635
==========

to

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Finnur, 2016-06-02 20:02:25]

Description was changed from

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Finnur, 2016-06-02 20:02:26]

finnur@chromium.org changed reviewers:
+ dschuyler@chromium.org

[dschuyler, 2016-06-02 22:18:15]

LGTM with a request to try timing them down.

https://codereview.chromium.org/2030203002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/icons.html (right):

https://codereview.chromium.org/2030203002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/icons.html:46: <g id="code"><path fill="none"
d="M0 0h24v24H0V0z"></path><path d="M9.4 16.6L4.8 12l4.6-4.6L8 6l-6 6 6 6
1.4-1.4zm5.2 0l4.6-4.6-4.6-4.6L16 6l6 6-6 6-1.4-1.4z"></path></g>
I believe we can remove these paths:
<path fill="none" d="M0 0h24v24H0V0z"></path>
from each of the new icons. If I read
the encoding correctly, it is just making
a square around the icon.

Please try removing them to see if I'm
correct.

[Finnur, 2016-06-03 00:47:11]

> Please try removing them to see if I'm correct.

Correct, as always. :)

[Finnur, 2016-06-03 00:47:39]

The CQ bit was checked by finnur@chromium.org

[Finnur, 2016-06-03 00:47:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
dschuyler@chromium.org
Link to the patchset: https://codereview.chromium.org/2030203002/#ps2 (title:
"Address feedback")

[commit-bot: I haz the power, 2016-06-03 00:48:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030203002/2
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2030203002/2

[commit-bot: I haz the power, 2016-06-03 02:36:39]

Description was changed from

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-03 02:36:40]

Committed patchset #2 (id:2)

[commit-bot: I haz the power, 2016-06-03 02:38:57]

Description was changed from

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Site Settings: Add icons for a few categories.

BUG=614277, 614277, 543635
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/8e1b4e7ae7d21f64f85dad5d35235fd443277cae
Cr-Commit-Position: refs/heads/master@{#397588}
==========

[commit-bot: I haz the power, 2016-06-03 02:38:58]

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