corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1311910 - Add telemetry to measure update error and update timeout rate for V2 and V4. r=francois,henry

Browse Source 
MozReview-Commit-ID: JL4aZrUOGH7
This commit is contained in:
DimiL
2016-12-19 09:43:02 +08:00
parent f9b2d6a99d
commit 3cb66c6aee
7 changed files with 56 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
toolkit/components/url-classifier/nsUrlClassifierStreamUpdater.cpp
View File

@@ -10,6 +10,7 @@
#include "nsIUploadChannel.h"
#include "nsIURI.h"
#include "nsIUrlClassifierDBService.h"
#include "nsUrlClassifierUtils.h"
#include "nsNetUtil.h"
#include "nsStreamUtils.h"
#include "nsStringStream.h"
@@ -643,11 +644,17 @@ nsUrlClassifierStreamUpdater::OnStartRequest(nsIRequest *request,
spec.get(), this));
}
nsCOMPtr<nsIUrlClassifierUtils> urlUtil =
do_GetService(NS_URLCLASSIFIERUTILS_CONTRACTID);
nsCString provider;
urlUtil->GetTelemetryProvider(mStreamTable, provider);
if (NS_FAILED(status)) {
// Assume we're overloading the server and trigger backoff.
downloadError = true;
mozilla::Telemetry::Accumulate(mozilla::Telemetry::URLCLASSIFIER_UPDATE_REMOTE_STATUS,
15 /* unknown response code */);
mozilla::Telemetry::Accumulate(mozilla::Telemetry::URLCLASSIFIER_UPDATE_REMOTE_STATUS2,
provider, 15 /* unknown response code */);
} else {
bool succeeded = false;
@@ -657,8 +664,8 @@ nsUrlClassifierStreamUpdater::OnStartRequest(nsIRequest *request,
uint32_t requestStatus;
rv = httpChannel->GetResponseStatus(&requestStatus);
NS_ENSURE_SUCCESS(rv, rv);
mozilla::Telemetry::Accumulate(mozilla::Telemetry::URLCLASSIFIER_UPDATE_REMOTE_STATUS,
HTTPStatusToBucket(requestStatus));
mozilla::Telemetry::Accumulate(mozilla::Telemetry::URLCLASSIFIER_UPDATE_REMOTE_STATUS2,
provider, HTTPStatusToBucket(requestStatus));
LOG(("nsUrlClassifierStreamUpdater::OnStartRequest %s (%d)", succeeded ?
"succeeded" : "failed", requestStatus));
if (!succeeded) {