MD Settings: fix setting-subpage icon color value.

MD Settings: fix setting-subpage icon color value.

BUG=654962
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2733083002
Cr-Commit-Position: refs/heads/master@{#455313}
Committed: https://chromium.googlesource.com/chromium/src/+/17218793515d69f9512895ff9b2e71f89d57982f

[scottchen, 2017-03-06 20:26:30]

Description was changed from

==========
MD Settings: fix setting-subpage icon color value.

BUG=654962
==========

to

==========
MD Settings: fix setting-subpage icon color value.

BUG=654962
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[scottchen, 2017-03-06 20:26:59]

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

[dschuyler, 2017-03-06 23:52:21]

https://codereview.chromium.org/2733083002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/settings_page/settings_subpage.html
(right):

https://codereview.chromium.org/2733083002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/settings_page/settings_subpage.html:17:
--iron-icon-fill-color: var(--paper-grey-600);
Will removing this line and letting the line in

https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/settin...

work here? If so, let's remove line 17.

[scottchen, 2017-03-07 22:04:53]

https://codereview.chromium.org/2733083002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/settings_page/settings_subpage.html
(right):

https://codereview.chromium.org/2733083002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/settings_page/settings_subpage.html:17:
--iron-icon-fill-color: var(--paper-grey-600);
On 2017/03/06 23:52:21, dschuyler wrote:
> Will removing this line and letting the line in
> 
>
https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/settin...
> 
> work here? If so, let's remove line 17.

err, not sure how the option to just remove it completely escaped me. Done.

[dschuyler, 2017-03-07 22:06:26]

lgtm

[scottchen, 2017-03-07 23:23:49]

The CQ bit was checked by scottchen@chromium.org

[commit-bot: I haz the power, 2017-03-07 23:24:24]

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, 2017-03-08 00:41:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488929029296700,
"parent_rev": "3273d29296a6ab2759363beb9a09ed43980f0af6", "commit_rev":
"17218793515d69f9512895ff9b2e71f89d57982f"}

[commit-bot: I haz the power, 2017-03-08 00:42:10]

Description was changed from

==========
MD Settings: fix setting-subpage icon color value.

BUG=654962
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: fix setting-subpage icon color value.

BUG=654962
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2733083002
Cr-Commit-Position: refs/heads/master@{#455313}
Committed:
https://chromium.googlesource.com/chromium/src/+/17218793515d69f9512895ff9b2e...
==========

[commit-bot: I haz the power, 2017-03-08 00:42:12]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/17218793515d69f9512895ff9b2e...