Closed Bug 1500300 Opened 7 years ago Closed 7 years ago

Scrolling the test-case for bug 1497959 crashes WebRender

Categories

(Core :: Graphics: WebRender, defect, P2)

Unspecified
All
defect

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox-esr60 --- unaffected
firefox63 --- unaffected
firefox64 --- disabled
firefox65 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug, )

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

https://crash-stats.mozilla.com/report/index/62212e4a-33e6-4c59-a63d-e7fec0181019 I'll take a look, but CCing some folks in case it's one of the crashes that you were hunting down.
Flags: needinfo?(emilio)
Potential workaround patch here - https://github.com/servo/webrender/pull/3219. I need nical or someone else familiar with blobs and tiling to check if it's a reasonable workaround in the interim.
Tangential note: I filed bug 1500401 to clean up the signature of the crash Emilio was running into.
> called `Option::unwrap()` on a `None` value
Crash Signature: [@ mozalloc_abort | abort | __rust_start_panic | rust_begin_unwind ] [@ webrender::resource_cache::ResourceCache::request_image ]
Priority: -- → P2
Depends on: 1500233
Keywords: crash
OS: Unspecified → All
Attached file GitHub Pull Request
Flags: needinfo?(emilio)
Blocks: 1500575
I need to write some tests for this.
Flags: needinfo?(emilio)
Actually I ended up writing a bunch of tiling test-cases in that WR pull request, and hitting tiled blob images is way harder now until they're enabled by default, which I believe Matt / Gankro are working on. So when that's done we should get a bunch of test-coverage for free.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(emilio)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: