yamaguchi@: Could you confirm? Thank you!

I confirmed the lock badge appears when connecting to a secure network but not when connecting to non-secured one. After signing in, the badge was not shown regardless of the wifi network type.
I think the right behavior is the one seen in OOBE screen as long unless the network is open (no password).

Based on Issue 874165, the right behavior is to show the badge (the lock icon beside WiFi icon) if the wireless network is encrypted.
The behavior on OOBE is correct, and the one after signed in is wrong.
To sgabriel@: please confirm if this is correct.

The badge (lock icon) has been shown only when the icon is in the dark color since more than 7 years ago. So my guess is that this become visible after we've changed the OOBE screen to bright color theme.

I believe that historically the "light" icons were used in the status area where we intentionally did not want to clutter the display with badge icons, and the "dark" icons were used in the system menu itself. We probably need to update that logic to explicitly specify where/when we want to badge the icons (which will be easier to do now that we are generating them as vector icons).

Thanks, that is good to know. After discussion we decided not to change the existing UI appearance. So we'd just make the icons on OOBE status tray same as in user session except for the color.

I wrote #7 to a wrong thread. Sorry.

According to Issue 874165 it seems the plan is to also show badges in tray icons.

stevenjb@, sgabriel@: WDYT?

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0062d38517b1eb6316fcee094fb4bf75163b0e39

commit 0062d38517b1eb6316fcee094fb4bf75163b0e39
Author: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Date: Wed Aug 22 13:23:19 2018

Hide lock icon in OOBE status tray wifi icon.

We have hidden the lock icon badge in the status tray to avoid visual
clutter, but the logic was not applied on dark color icons in OOBE.


Test: manual test by connecting to an encrypted wifi access point at OOBE
Bug:  874012 
Change-Id: I529591faeba8febfce918dd400cc3d230da58658
Reviewed-on: https://chromium-review.googlesource.com/1177081
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584995}
[modify] https://crrev.com/0062d38517b1eb6316fcee094fb4bf75163b0e39/ash/system/network/network_icon.cc

re #9 as discussed during last week meeting, let's not display the lock on wifi in status tray.

#12: So the spec in Issue 874165 is changed, right?

Yes, I've updated the spec accordingly.