[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current location,
     verify Net.WiFi histograms and AccessPointData.

Committed: https://crrev.com/30bcd45ac9f5875bdcf7a17020b4fe7d94d503bf
Cr-Commit-Position: refs/heads/master@{#430326}

[Robert Sesek, 2016-11-01 18:56:25]

The CQ bit was checked by rsesek@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-01 18:56:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Robert Sesek, 2016-11-01 18:58:24]

Description was changed from

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
==========

to

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location, verify Net.WiFi histograms and AccessPointData.
==========

[Robert Sesek, 2016-11-01 18:58:24]

rsesek@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Robert Sesek, 2016-11-01 18:58:35]

Description was changed from

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location, verify Net.WiFi histograms and AccessPointData.
==========

to

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location,
     verify Net.WiFi histograms and AccessPointData.
==========

[commit-bot: I haz the power, 2016-11-01 19:57:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-01 19:57:41]

Dry run: This issue passed the CQ dry run.

[Robert Sesek, 2016-11-04 14:06:49]

Friendly ping.

[Robert Sesek, 2016-11-04 20:15:46]

rsesek@chromium.org changed reviewers:
+ mcasas@chromium.org

[Robert Sesek, 2016-11-04 20:15:47]

+mcasas

[mcasas, 2016-11-04 21:45:52]

lgtm

https://codereview.chromium.org/2472463002/diff/1/device/geolocation/wifi_dat...
File device/geolocation/wifi_data_provider_mac.mm (right):

https://codereview.chromium.org/2472463002/diff/1/device/geolocation/wifi_dat...
device/geolocation/wifi_data_provider_mac.mm:109: const int
kDefaultPollingInterval = 120000;                // 2 mins
micro-nit here and in the following lines:
kDefaultPollingInterval = 120 * base::Time::kMillisecondsPerSecond; 
?
Might be too verbose for these lines though. Your call.

[Robert Sesek, 2016-11-04 22:05:13]

https://codereview.chromium.org/2472463002/diff/1/device/geolocation/wifi_dat...
File device/geolocation/wifi_data_provider_mac.mm (right):

https://codereview.chromium.org/2472463002/diff/1/device/geolocation/wifi_dat...
device/geolocation/wifi_data_provider_mac.mm:109: const int
kDefaultPollingInterval = 120000;                // 2 mins
On 2016/11/04 21:45:52, mcasas wrote:
> micro-nit here and in the following lines:
> kDefaultPollingInterval = 120 * base::Time::kMillisecondsPerSecond; 
> ?
> Might be too verbose for these lines though. Your call.

These were from the old file
(https://codereview.chromium.org/2472463002/diff/1/device/geolocation/wifi_dat...),
so I left them as-is.

[Robert Sesek, 2016-11-07 17:26:55]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2016-11-07 17:27:16]

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, 2016-11-07 18:51:03]

Description was changed from

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location,
     verify Net.WiFi histograms and AccessPointData.
==========

to

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location,
     verify Net.WiFi histograms and AccessPointData.
==========

[commit-bot: I haz the power, 2016-11-07 18:51:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-07 18:56:47]

Description was changed from

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location,
     verify Net.WiFi histograms and AccessPointData.
==========

to

==========
[Mac] Use CoreWLAN directly now that macOS 10.9 is the minimum version.

BUG=606408
TEST=With WiFi turned on, go to https://maps.google.com, update current
location,
     verify Net.WiFi histograms and AccessPointData.

Committed: https://crrev.com/30bcd45ac9f5875bdcf7a17020b4fe7d94d503bf
Cr-Commit-Position: refs/heads/master@{#430326}
==========

[commit-bot: I haz the power, 2016-11-07 18:56:48]

Patchset 1 (id:??) landed as
https://crrev.com/30bcd45ac9f5875bdcf7a17020b4fe7d94d503bf
Cr-Commit-Position: refs/heads/master@{#430326}

[Michael van Ouwerkerk, 2016-11-08 10:20:03]

lgtm