WiFi hotspot is listed instead of Tether network in System Tray |
||||
Issue descriptionChrome Version: 61.0.3120.0 OS: Chrome What steps will reproduce the problem? (1) Connect to Tether network. (2) If Ethernet is plugged in, unplug it. (3) Open System Tray. What is the expected result? "Connected to" + the name of the active host should appear, along with the Cellular bars icon showing signal strength. What happens instead? The name of the underlying WiFi hotspot appears, along with WiFi arcs showing signal strength. Please use labels and text to provide additional information. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Jun 14 2017
,
Jun 21 2017
,
Jun 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5a0eba4b0089554977fde98d3a30d3ca74a750a0 commit 5a0eba4b0089554977fde98d3a30d3ca74a750a0 Author: khorimoto <khorimoto@chromium.org> Date: Thu Jun 22 19:17:00 2017 [CrOS Tether] Sort Tether network lists. With this change, Tether networks in the networking UI (i.e., settings and quick settings) are sorted in the same order that the Tether component prioritizes them when performing a host scan. The sorting rules are as follows: (1) The device which has most recently sent a successful ConnectTetheringResponse is always at the front of the queue. (2) Devices which have most recently sent a successful TetherAvailabilityResponse are next in the order, as long as they do not violate the first rule. (3) All other devices are left in the order they are passed. This CL also updates the GetNetworkListByType() logic in NetworkStateHandler; after this change, all connected/connecting networks are listed before non-active networks, and Wi-Fi networks associated with Tether networks are no longer returned since they are supposed to be obscured behind their associated Tether networks. BUG=672263, 729796 , 729801 Review-Url: https://codereview.chromium.org/2945643002 Cr-Commit-Position: refs/heads/master@{#481621} [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/BUILD.gn [delete] https://crrev.com/acdd2a4349f0316bcc5f6c09a5fce64d92e82aaa/chromeos/components/tether/host_scan_device_prioritizer.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scan_device_prioritizer.h [add] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scan_device_prioritizer_impl.cc [add] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scan_device_prioritizer_impl.h [add] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scan_device_prioritizer_impl_unittest.cc [delete] https://crrev.com/acdd2a4349f0316bcc5f6c09a5fce64d92e82aaa/chromeos/components/tether/host_scan_device_prioritizer_unittest.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scanner_operation_unittest.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/host_scanner_unittest.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/initializer.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/components/tether/initializer.h [delete] https://crrev.com/acdd2a4349f0316bcc5f6c09a5fce64d92e82aaa/chromeos/components/tether/mock_host_scan_device_prioritizer.cc [delete] https://crrev.com/acdd2a4349f0316bcc5f6c09a5fce64d92e82aaa/chromeos/components/tether/mock_host_scan_device_prioritizer.h [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/network/network_state_handler.cc [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/network/network_state_handler.h [modify] https://crrev.com/5a0eba4b0089554977fde98d3a30d3ca74a750a0/chromeos/network/network_state_handler_unittest.cc
,
Jun 22 2017
,
Jan 22 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by omrilio@chromium.org
, Jun 9 2017