Commit 03fc9d4a authored by Andre Le's avatar Andre Le Committed by Chromium LUCI CQ

[CrOS PhoneHub] Fix crash in CrOS Linux build.

This change fix the Linux build crash caused by a DCHECK in
MetricsRecorder::OnConnectionStatusChanged()

Bug: 1106937
Change-Id: Id8b1f7f5c0bd414fa55b44234994e9c815e77750
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2574622Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Andre Le <leandre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833873}
parent f68f4ab8
...@@ -53,8 +53,7 @@ void ConnectionManagerImpl::MetricsRecorder::OnConnectionStatusChanged() { ...@@ -53,8 +53,7 @@ void ConnectionManagerImpl::MetricsRecorder::OnConnectionStatusChanged() {
case ConnectionManager::Status::kDisconnected: case ConnectionManager::Status::kDisconnected:
if (prev_status == ConnectionManager::Status::kConnected) { if (prev_status == ConnectionManager::Status::kConnected) {
UMA_HISTOGRAM_TIMES("PhoneHub.Connectivity.Duration", delta); UMA_HISTOGRAM_TIMES("PhoneHub.Connectivity.Duration", delta);
} else { } else if (prev_status == ConnectionManager::Status::kConnecting) {
DCHECK(prev_status == ConnectionManager::Status::kConnecting);
RecordConnectionSuccessMetric(false); RecordConnectionSuccessMetric(false);
} }
break; break;
......
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