QA/Platform/Graphics/Features/Discard Animated Images
Jump to navigation
Jump to search
People
| Name | Role | Contact |
|---|---|---|
| Timothy Nikkel | Development | IRC: tnikkel |
| Anthony Hughes | Quality Assurance | IRC: ashughes |
| Milan Sreckovic | Manager | IRC: milan |
Bugs
Milestones
| Milestone | Dates | Deliverables | Status |
|---|---|---|---|
| M1 (Nightly 55) | March 6 - April 18, 2017 |
|
[ON TRACK] |
Risk Analysis
Memory
| Risk | Mitigation | Verification | Status |
|---|---|---|---|
| MEMORY_IMAGES_CONTENT_USED_UNCOMPRESSED lower than expected | Reduce impact to zero via bug 1347042 | Will want to monitor via Telemetry | [ON TRACK] |
Web Compatibility
| Risk | Mitigation | Verification | Status |
|---|---|---|---|
| Animated images not displaying because they haven't finished decoding | Manual testing and monitoring public feedback channels | No blocker bugs prior to uplift |
Performance
| Risk (list any risks that need to be mitigated) |
Mitigation (automation, manual testing, a/b testing, etc) |
Verification (how/when to decide risk mitigated) |
Status |
|---|---|---|---|
| IMAGE_DECODE* probes aren't specific to animated images | Landing probes specific to animated images via bug 1347302 | Will need to monitor via Telemetry to establish a baseline |
Stability
| Risk | Mitigation | Verification | Status |
|---|---|---|---|
| Increased instability | Monitor stability through Socorro & Telemetry | Crash rates should decrease, particularly OOMs |
Hardware Compatibility
| Risk | Mitigation | Verification | Status |
|---|---|---|---|
| Excessive decode latency on low-end systems | Profile low-end systems via Telemetry |
Experiments
- Nightly 55
Intent will be to do an A/B comparison of the following metrics with the pref flipped:
- IMAGE_DECODE_COUNT: https://mzl.la/2mDjTxY (seems pretty stable)
- IMAGE_DECODE_TIME: https://mzl.la/2mDfv1O (seems pretty stable)
- IMAGE_DECODE_ON_DRAW_LATENCY: https://mzl.la/2mDjPyh (trending down)
- MEMORY_IMAGES_CONTENT_USED_UNCOMPRESSED: https://mzl.la/2mDg2AF (flatlined at 0 -- bug?)
- Overall crash rates via SQL Telemetry
- Crash signature comparison via Socorro ("images" in signature, proto_signature, and topmost filename)
- https://codepen.io/anon/pen/jBwvow?&editors=111
Risks:
- The above probes don't discriminate between animated and static images. As a result the migration of numbers for animated images may be less visible at best or invisible at worst. We need to find out from Telemetry team how long it'd take to establish a baseline with new probes specific for animated images. (Owner: ashughes) -- this is being addressed via bug 1347302