Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-29 15:52:07 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 687858. Avoid accumulating telemetry for size decodes. r=jlebar

Browse Source 
These just add a bunch of noise to the data because they break some of the assumptions we have.
Currently this doesn't have much impact because decode on draw is disabled.

--HG--
extra : rebase_source : 1022ad9bd43ea5f8be342c05ab857bb44ed7e59c
This commit is contained in:
Jeff Muizelaar 2011-09-21 12:19:53 -04:00
parent a4b1332fbc
commit 4e1e4e8155
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
modules/libpr0n/src/RasterImage.cpp
View File

@ -2753,7 +2753,7 @@ imgDecodeWorker::Run()
}
TimeDuration decodeLatency = TimeStamp::Now() - start;
if (chunkCount) {
if (chunkCount && !image->mDecoder->IsSizeDecode()) {
Telemetry::Accumulate(Telemetry::IMAGE_DECODE_LATENCY, PRInt32(decodeLatency.ToMicroseconds()));
Telemetry::Accumulate(Telemetry::IMAGE_DECODE_CHUNKS, chunkCount);
}
@ -2772,7 +2772,11 @@ imgDecodeWorker::Run()
// If the decode finished, shutdown the decoder
if (image->mDecoder && image->IsDecodeFinished()) {
Telemetry::Accumulate(Telemetry::IMAGE_DECODE_TIME, PRInt32(mDecodeTime.ToMicroseconds()));
if (!image->mDecoder->IsSizeDecode()) {
Telemetry::Accumulate(Telemetry::IMAGE_DECODE_TIME, PRInt32(mDecodeTime.ToMicroseconds()));
}
rv = image->ShutdownDecoder(RasterImage::eShutdownIntent_Done);
if (NS_FAILED(rv)) {
image->DoError();