Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(938)

Side by Side Diff: device/geolocation/wifi_data_provider_common_win.cc

Issue 2200483002: Geolocation cleanup: run clang-format (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "device/geolocation/wifi_data_provider_common_win.h" 5 #include "device/geolocation/wifi_data_provider_common_win.h"
6 6
7 #include <assert.h> 7 #include <assert.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
11 #include "device/geolocation/wifi_data_provider_common.h" 11 #include "device/geolocation/wifi_data_provider_common.h"
12 12
13 namespace device { 13 namespace device {
14 14
15 bool ConvertToAccessPointData(const NDIS_WLAN_BSSID& data, 15 bool ConvertToAccessPointData(const NDIS_WLAN_BSSID& data,
16 AccessPointData *access_point_data) { 16 AccessPointData* access_point_data) {
17 // Currently we get only MAC address, signal strength and SSID. 17 // Currently we get only MAC address, signal strength and SSID.
18 // TODO(steveblock): Work out how to get age, channel and signal-to-noise. 18 // TODO(steveblock): Work out how to get age, channel and signal-to-noise.
19 DCHECK(access_point_data); 19 DCHECK(access_point_data);
20 access_point_data->mac_address = MacAddressAsString16(data.MacAddress); 20 access_point_data->mac_address = MacAddressAsString16(data.MacAddress);
21 access_point_data->radio_signal_strength = data.Rssi; 21 access_point_data->radio_signal_strength = data.Rssi;
22 // Note that _NDIS_802_11_SSID::Ssid::Ssid is not null-terminated. 22 // Note that _NDIS_802_11_SSID::Ssid::Ssid is not null-terminated.
23 base::UTF8ToUTF16(reinterpret_cast<const char*>(data.Ssid.Ssid), 23 base::UTF8ToUTF16(reinterpret_cast<const char*>(data.Ssid.Ssid),
24 data.Ssid.SsidLength, 24 data.Ssid.SsidLength, &access_point_data->ssid);
25 &access_point_data->ssid);
26 return true; 25 return true;
27 } 26 }
28 27
29 int GetDataFromBssIdList(const NDIS_802_11_BSSID_LIST& bss_id_list, 28 int GetDataFromBssIdList(const NDIS_802_11_BSSID_LIST& bss_id_list,
30 int list_size, 29 int list_size,
31 WifiData::AccessPointDataSet* data) { 30 WifiData::AccessPointDataSet* data) {
32 // Walk through the BSS IDs. 31 // Walk through the BSS IDs.
33 int found = 0; 32 int found = 0;
34 const uint8_t* iterator = 33 const uint8_t* iterator =
35 reinterpret_cast<const uint8_t*>(&bss_id_list.Bssid[0]); 34 reinterpret_cast<const uint8_t*>(&bss_id_list.Bssid[0]);
36 const uint8_t* end_of_buffer = 35 const uint8_t* end_of_buffer =
37 reinterpret_cast<const uint8_t*>(&bss_id_list) + list_size; 36 reinterpret_cast<const uint8_t*>(&bss_id_list) + list_size;
38 for (int i = 0; i < static_cast<int>(bss_id_list.NumberOfItems); ++i) { 37 for (int i = 0; i < static_cast<int>(bss_id_list.NumberOfItems); ++i) {
39 const NDIS_WLAN_BSSID *bss_id = 38 const NDIS_WLAN_BSSID* bss_id =
40 reinterpret_cast<const NDIS_WLAN_BSSID*>(iterator); 39 reinterpret_cast<const NDIS_WLAN_BSSID*>(iterator);
41 // Check that the length of this BSS ID is reasonable. 40 // Check that the length of this BSS ID is reasonable.
42 if (bss_id->Length < sizeof(NDIS_WLAN_BSSID) || 41 if (bss_id->Length < sizeof(NDIS_WLAN_BSSID) ||
43 iterator + bss_id->Length > end_of_buffer) { 42 iterator + bss_id->Length > end_of_buffer) {
44 break; 43 break;
45 } 44 }
46 AccessPointData access_point_data; 45 AccessPointData access_point_data;
47 if (ConvertToAccessPointData(*bss_id, &access_point_data)) { 46 if (ConvertToAccessPointData(*bss_id, &access_point_data)) {
48 data->insert(access_point_data); 47 data->insert(access_point_data);
49 ++found; 48 ++found;
50 } 49 }
51 // Move to the next BSS ID. 50 // Move to the next BSS ID.
52 iterator += bss_id->Length; 51 iterator += bss_id->Length;
53 } 52 }
54 return found; 53 return found;
55 } 54 }
56 55
57 } // namespace device 56 } // namespace device
OLDNEW
« no previous file with comments | « device/geolocation/wifi_data_provider_common_unittest.cc ('k') | device/geolocation/wifi_data_provider_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698