MD Settings: Use shared cr_elements iconset when possible

MD Settings: Use shared cr_elements iconset when possible

This is the first step in getting Settings off of iron-icons. Changes any
instances of icons already in the 'cr' iconset to use that iconset.

A follow-up will consolidate the rest of the Settings icons inside a custom
Settings iconset.

R=dbeam@chromium.org
BUG=605821
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/bf2d0f7b9eeeab5618f0656e81d40aa5d736557d
Cr-Commit-Position: refs/heads/master@{#392599}

[michaelpg, 2016-05-09 19:39:15]

Description was changed from

==========
MD Settings: Use shared cr_elements iconset when possible

This is the first step in getting Settings off of iron-icons. Changes any
instances of icons already in the 'cr' iconset to use that iconset.

A follow-up will consolidate the rest of the Settings icons inside a custom
Settings iconset.

R=dbeam@chromium.org
BUG=605821
==========

to

==========
MD Settings: Use shared cr_elements iconset when possible

This is the first step in getting Settings off of iron-icons. Changes any
instances of icons already in the 'cr' iconset to use that iconset.

A follow-up will consolidate the rest of the Settings icons inside a custom
Settings iconset.

R=dbeam@chromium.org
BUG=605821
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[michaelpg, 2016-05-09 19:40:47]

Dan, PTAL. It's a pretty boring change. I'll send the next one to stevenjb.

[Dan Beam, 2016-05-09 19:54:05]

lgtm

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

https://codereview.chromium.org/1960323002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/settings_shared_css.html:54:
iron-icon[icon=done] {
"i wish i had a 'cr:'!" - done

[michaelpg, 2016-05-09 20:53:48]

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

https://codereview.chromium.org/1960323002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/settings_shared_css.html:54:
iron-icon[icon=done] {
On 2016/05/09 19:54:05, Dan Beam wrote:
> "i wish i had a 'cr:'!" - done

Parsing as "this needs a 'cr:' prefix", no it doesn't, because done is still
being pulled from iron-icons.html (see languages_page.html).

Parsing as "this should be in the cr_elements/icon.html", I'll add it there when
other UIs use it.

[michaelpg, 2016-05-10 04:35:41]

The CQ bit was checked by michaelpg@chromium.org

[michaelpg, 2016-05-10 04:35:42]

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

[commit-bot: I haz the power, 2016-05-10 04:35:49]

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

[michaelpg, 2016-05-10 04:40:19]

Patchset #3 (id:40001) has been deleted

[michaelpg, 2016-05-10 14:32:55]

The CQ bit was checked by michaelpg@chromium.org

[michaelpg, 2016-05-10 14:32:55]

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

[commit-bot: I haz the power, 2016-05-10 14:33:31]

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

[commit-bot: I haz the power, 2016-05-10 14:37:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-10 14:40:12]

Description was changed from

==========
MD Settings: Use shared cr_elements iconset when possible

This is the first step in getting Settings off of iron-icons. Changes any
instances of icons already in the 'cr' iconset to use that iconset.

A follow-up will consolidate the rest of the Settings icons inside a custom
Settings iconset.

R=dbeam@chromium.org
BUG=605821
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Use shared cr_elements iconset when possible

This is the first step in getting Settings off of iron-icons. Changes any
instances of icons already in the 'cr' iconset to use that iconset.

A follow-up will consolidate the rest of the Settings icons inside a custom
Settings iconset.

R=dbeam@chromium.org
BUG=605821
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/bf2d0f7b9eeeab5618f0656e81d40aa5d736557d
Cr-Commit-Position: refs/heads/master@{#392599}
==========

[commit-bot: I haz the power, 2016-05-10 14:40:13]

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