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

Bug 1607769 - Remove script loader telemetry added by Bug 1579876 and Bug 1573904. r=jorendorff

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D60288
This commit is contained in:
Nicolas B. Pierron
2020-02-03 18:55:44 +00:00
parent c040db93ec
commit a77a83751a
10 changed files with 6 additions and 199 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
dom/script/ScriptLoadHandler.cpp
View File

@@ -70,25 +70,6 @@ nsresult ScriptLoadHandler::DecodeRawDataHelper(const uint8_t* aData,
MakeSpan(scriptText.begin() + haveRead, needed.value()), aEndOfStream);
MOZ_ASSERT(written <= needed.value());
// Telemetry: Measure the throughput at which bytes are streamed and the ratio
// of streamed bytes, such that we can determine the effectiveness of a
// streaming parser for JavaScript.
using namespace mozilla::Telemetry;
if (aEndOfStream && haveRead) {
// Compute the percent of data transfered incrementally.
Accumulate(DOM_SCRIPT_LOAD_INCREMENTAL_RATIO,
100 * haveRead / (haveRead + written));
// Compute the rate of transfer of the incremental data calls averaged
// across the time needed to complete the request.
auto streamingTime = TimeStamp::Now() - mFirstOnIncrementalData;
double ms = streamingTime.ToMilliseconds();
Accumulate(DOM_SCRIPT_LOAD_INCREMENTAL_AVG_TRANSFER_RATE, haveRead / ms);
mRequest->mStreamingTime = streamingTime;
}
if (!aEndOfStream && !haveRead) {
mFirstOnIncrementalData = TimeStamp::Now();
}
haveRead += written;
MOZ_ASSERT(haveRead <= capacity.value(),
"mDecoder produced more data than expected");