Commit ab1b7842 authored by Henrik Grunell's avatar Henrik Grunell Committed by Commit Bot

Fix network error code sign for WebRTC text log uploads.

Error codes are apparently negative, whereas the histogram enum is
defined with positive values.

Bug: 915679
Change-Id: Ie14569dcb1283b5960f11c94321863537238dae7
Reviewed-on: https://chromium-review.googlesource.com/c/1473170Reviewed-by: default avatarTommi <tommi@chromium.org>
Commit-Queue: Henrik Grunell <grunell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634683}
parent f54db458
......@@ -5,6 +5,7 @@
#include "chrome/browser/media/webrtc/webrtc_log_uploader.h"
#include <stddef.h>
#include <cstdlib>
#include <utility>
#include "base/bind.h"
......@@ -669,7 +670,7 @@ void WebRtcLogUploader::NotifyUploadDoneAndLogStats(
"WebRtcTextLogging.UploadFailureReason",
WebRtcLoggingHandlerHost::UploadFailureReason::kNetworkError);
base::UmaHistogramSparse("WebRtcTextLogging.UploadFailureNetErrorCode",
network_error_code);
std::abs(network_error_code));
}
error_message =
base::StrCat({"Uploading failed, response code: ",
......
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