New issue
Advanced search Search tips

Issue 713378 link

Starred by 1 user
Status: Fixed
Owner:
btolsch@chromium.org
Closed: Jul 2017
Cc:
mfo...@chromium.org
btolsch@chromium.org
powerb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

Mac needs a source for Wifi SSID

Project Member Reported by btolsch@chromium.org, Apr 19 2017 
In order to help identify connected networks for discovery, the DiscoveryNetworkMonitor ( crbug.com/698943  crrev.com/2750453002/) needs a way to get the SSID for connected Wifi interfaces.  While net::GetWifiSSID provides one form of this, it isn't implemented for Mac.  wifi::WifiService::GetConnectedNetworkSSID provides the same functionality as net::GetWifiSSID but requires first creating a WifiService object.

It would be preferable to get the Wifi SSID by interface (i.e. allow multiple Wifi interfaces to be up, with different associated SSIDs).
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 7 2017 

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

commit bf4426c77462169c84c817eeae03a9cd7169672a
Author: btolsch <btolsch@chromium.org>
Date: Fri Jul 07 04:16:30 2017

Add Mac implementation of GetDiscoveryNetworkInfoList

This change enables DiscoveryNetworkMonitor to work on Mac.  Some small
syntax changes are made to the _posix.cc file and an implementation for
getting the WiFi SSID associated with an interface is also added.

Bug:  698943 ,  713378 
Change-Id: I1f50bbe7169e5ac017dba9ee24b3f84a24d4aecd
Reviewed-on: https://chromium-review.googlesource.com/527952
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Commit-Queue: Brandon Tolsch <btolsch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484828}
[modify] https://crrev.com/bf4426c77462169c84c817eeae03a9cd7169672a/chrome/browser/media/router/discovery/BUILD.gn
[modify] https://crrev.com/bf4426c77462169c84c817eeae03a9cd7169672a/chrome/browser/media/router/discovery/discovery_network_list_posix.cc
[add] https://crrev.com/bf4426c77462169c84c817eeae03a9cd7169672a/chrome/browser/media/router/discovery/discovery_network_list_wifi_mac.mm

Comment 2 by btolsch@chromium.org, Jul 7 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment