Attaching Expected video for reference

Deleted: Expected_NoWi-FiNetworks.mp4 14.0 MB

	Expected_NoWi-FiNetworks.mp4 14.0 MB View Download

This delay definitely seems longer in M65 than it does in M63. @zork, could we set it to hide the list instantly when the toggle is switched off?

That seems like a reasonable solution.

Daisy, could you have someone look into this?

xiaoyinh@, could this issue be related to your change?

My recent CL to change the update frequency is for the bluetooth tray. This bug is about the network list, so it's probably irrelevant. Looking at the code, it feels like it takes sometime to disable the wifi and receive a networkStateChanged event. 

I think we can fix this by hide the network list immediately when the toggle is off. 

Thanks! If it's a simple fix, could you help take a look at it since you've been very familiar with this part of codes?

Sure. Assigning to myself.

Any update on this?

This is marked as release block stable for 65, so we need a fix in the next couple weeks.

Sorry I've been busy with other tasks, will take a close look today.

Somehow I couldn't repro on ToT: the label "Wi-Fi is turned off" shows after ~1s without any Wi-Fi networks. 

Looks like 1s comes from the update frequency in TrayNetworkStateObserver.
I've uploaded a CL to update immediately when wifi becomes disabled. 
In addition, we don't need to show wifi network list when it's disabled.

https://chromium-review.googlesource.com/c/chromium/src/+/936003

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

commit 50ef07e85110f261574c300e8c236274ea05797f
Author: Sarah Hu <xiaoyinh@chromium.org>
Date: Mon Feb 26 22:26:38 2018

Update wifi network immediately when it's disabled

Bug:  798325 
Change-Id: Icc0e4a8e6e61b01e5736d52e65632b3e47a1f0d6
Reviewed-on: https://chromium-review.googlesource.com/936003
Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539290}
[modify] https://crrev.com/50ef07e85110f261574c300e8c236274ea05797f/ash/system/network/network_list.cc
[modify] https://crrev.com/50ef07e85110f261574c300e8c236274ea05797f/ash/system/network/tray_network_state_observer.cc
[modify] https://crrev.com/50ef07e85110f261574c300e8c236274ea05797f/ash/system/network/tray_network_state_observer.h

This bug requires manual review: We are only 7 days from stable.
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

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

commit 5dfbcb4816fccd178cef5ab6a70fa3a99e6c57ef
Author: Sarah Hu <xiaoyinh@chromium.org>
Date: Wed Feb 28 02:47:28 2018

[Merge to M65]Update wifi network immediately when it's disabled

TBR=stevenjb@chromium.org
(cherry picked from commit 50ef07e85110f261574c300e8c236274ea05797f)

Bug:  798325 
Change-Id: Icc0e4a8e6e61b01e5736d52e65632b3e47a1f0d6
Reviewed-on: https://chromium-review.googlesource.com/936003
Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#539290}
Reviewed-on: https://chromium-review.googlesource.com/940767
Reviewed-by: Xiaoyin Hu <xiaoyinh@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#621}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/5dfbcb4816fccd178cef5ab6a70fa3a99e6c57ef/ash/system/network/network_list.cc
[modify] https://crrev.com/5dfbcb4816fccd178cef5ab6a70fa3a99e6c57ef/ash/system/network/tray_network_state_observer.cc
[modify] https://crrev.com/5dfbcb4816fccd178cef5ab6a70fa3a99e6c57ef/ash/system/network/tray_network_state_observer.h