Commit 2b9159c0 authored by Tarun Bansal's avatar Tarun Bansal Committed by Commit Bot

Replace DCHECK by DEBUG warning in NetworkMetricsProvider.

This code is quite old, affects only ChromeOS,
we really do not know who is the actual owner of this code,
or how these metrics are being used.

Change-Id: Iac7f8c24f37aa10e2f5053e0e2c734fa6220f745
Bug: 801884
Reviewed-on: https://chromium-review.googlesource.com/1034455
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: default avatarMark Pearson <mpearson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554891}
parent 9c66e77a
...@@ -404,10 +404,11 @@ void NetworkMetricsProvider::WriteWifiAccessPointProto( ...@@ -404,10 +404,11 @@ void NetworkMetricsProvider::WriteWifiAccessPointProto(
for (const base::StringPiece& oui_str : base::SplitStringPiece( for (const base::StringPiece& oui_str : base::SplitStringPiece(
info.oui_list, " ", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) { info.oui_list, " ", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL)) {
uint32_t oui; uint32_t oui;
if (base::HexStringToUInt(oui_str, &oui)) if (base::HexStringToUInt(oui_str, &oui)) {
vendor->add_element_identifier(oui); vendor->add_element_identifier(oui);
else } else {
NOTREACHED(); DLOG(WARNING) << "Error when parsing OUI list of the WiFi access point";
}
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment