Closed
Bug 687858
Opened 13 years ago
Closed 13 years ago
Avoid measuring telemetry for size decodes
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: jrmuizel, Assigned: jrmuizel)
Details
Attachments
(1 file)
1.72 KB,
patch
|
justin.lebar+bug
:
review+
|
Details | Diff | Splinter Review |
These just add a bunch of noise to the data because they break some of the assumptions we have.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #561200 -
Flags: review?(justin.lebar+bug)
Comment 2•13 years ago
|
||
Comment on attachment 561200 [details] [diff] [review] Avoid accumulating telemetry for size decodes r=me, but can we clear the existing telemetry data before we land this, so the histogram isn't further muddled?
Attachment #561200 -
Flags: review?(justin.lebar+bug) → review+
Comment 3•13 years ago
|
||
Taras, how would we go about invalidating the existing data for this histogram?
Assignee | ||
Comment 4•13 years ago
|
||
(In reply to Justin Lebar [:jlebar] from comment #3) > Taras, how would we go about invalidating the existing data for this > histogram? In this case, I don't think we need to invalidate the data because we're not doing size decodes yet because decode on draw is not on. Still, I'm interested in the answer.
Comment 5•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/518c3a56a5da
Assignee: nobody → jmuizelaar
Status: NEW → RESOLVED
Closed: 13 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
Version: unspecified → Trunk
Comment 6•13 years ago
|
||
Would it be useful to measure it but measure it separately?
You need to log in
before you can comment on or make changes to this bug.
Description
•