MD Settings: Internet: Fix secure icon and badge colors

MD Settings: Internet: Fix secure icon and badge colors

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

Review-Url: https://codereview.chromium.org/2813193003
Cr-Commit-Position: refs/heads/master@{#464240}
Committed: https://chromium.googlesource.com/chromium/src/+/a83687c6c41879be7fb501972457d55bfe1ee2af

[stevenjb, 2017-04-12 19:26:00]

Description was changed from

==========
MD Settings: Internet: Fix secure icon and badge colors

BUG=704008
==========

to

==========
MD Settings: Internet: Fix secure icon and badge colors

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

[stevenjb, 2017-04-12 19:27:09]

stevenjb@chromium.org changed reviewers:
+ michaelpg@chromium.org

[michaelpg, 2017-04-12 23:57:30]

lgtm but the converted numbers are wacky, we usually round those (see below)

https://codereview.chromium.org/2813193003/diff/1/ui/webui/resources/cr_eleme...
File ui/webui/resources/cr_elements/network/network_icons.html (right):

https://codereview.chromium.org/2813193003/diff/1/ui/webui/resources/cr_eleme...
ui/webui/resources/cr_elements/network/network_icons.html:28: <path
d="M1,3.99980749 C1,3.44762906 1.45097518,3 1.99077797,3 L6.00922203,3
C6.55641359,3 7,3.44371665 7,3.99980749 L7,7.00019251 C7,7.55237094 6.54902482,8
6.00922203,8 L1.99077797,8 C1.44358641,8 1,7.55628335 1,7.00019251 L1,3.99980749
Z M3.5,5 L4.5,5 L4.5,6 L3.5,6 L3.5,5 Z"></path>
please tidy up these numbers.

If you temporarily remove the <defs> tags, you can pasting the <svg>...</svg>
markup into https://jakearchibald.github.io/svgomg/ to get the simplified
<paths> and replace those here.

[stevenjb, 2017-04-13 00:08:20]

https://codereview.chromium.org/2813193003/diff/1/ui/webui/resources/cr_eleme...
File ui/webui/resources/cr_elements/network/network_icons.html (right):

https://codereview.chromium.org/2813193003/diff/1/ui/webui/resources/cr_eleme...
ui/webui/resources/cr_elements/network/network_icons.html:28: <path
d="M1,3.99980749 C1,3.44762906 1.45097518,3 1.99077797,3 L6.00922203,3
C6.55641359,3 7,3.44371665 7,3.99980749 L7,7.00019251 C7,7.55237094 6.54902482,8
6.00922203,8 L1.99077797,8 C1.44358641,8 1,7.55628335 1,7.00019251 L1,3.99980749
Z M3.5,5 L4.5,5 L4.5,6 L3.5,6 L3.5,5 Z"></path>
On 2017/04/12 23:57:30, michaelpg wrote:
> please tidy up these numbers.
> 
> If you temporarily remove the <defs> tags, you can pasting the <svg>...</svg>
> markup into https://jakearchibald.github.io/svgomg/ to get the simplified
> <paths> and replace those here.

Oops, thought I was using the svgomg'd output, but I was having problems with
the fill data (we need to keep fill-rule="evenodd" but elim fill="#FFF"), so
went back and forth and ended up with the wrong data, thanks for catching.

[stevenjb, 2017-04-13 00:08:23]

The CQ bit was checked by stevenjb@chromium.org

[stevenjb, 2017-04-13 00:08:23]

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

[commit-bot: I haz the power, 2017-04-13 00:09:31]

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-04-13 01:55:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492042103042550,
"parent_rev": "a95ba76f368fba30b23363b7692cae2da92cb062", "commit_rev":
"a83687c6c41879be7fb501972457d55bfe1ee2af"}

[commit-bot: I haz the power, 2017-04-13 01:56:31]

Description was changed from

==========
MD Settings: Internet: Fix secure icon and badge colors

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

to

==========
MD Settings: Internet: Fix secure icon and badge colors

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

Review-Url: https://codereview.chromium.org/2813193003
Cr-Commit-Position: refs/heads/master@{#464240}
Committed:
https://chromium.googlesource.com/chromium/src/+/a83687c6c41879be7fb501972457...
==========

[commit-bot: I haz the power, 2017-04-13 01:56:31]

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