Closed Bug 756813 Opened 13 years ago Closed 13 years ago

crash in mozilla::layers::ImageContainer::GetCurrentSize

Categories

(Core :: Graphics: Layers, defect)

defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: gal, Assigned: dzbarsky)

Details

(Keywords: crash)

Crash Data

Attachments

(1 file, 2 obsolete files)

This bug was filed from the Socorro interface and is report bp-e36766b9-17d2-4593-89b6-ecaa12120519 . ============================================================= We have to do state.mImageRenderer.GetContainer() early in nsDisplayBackground::TryOptimizeToImageLayer and if it returns null, don't optimize into a layer. This crash only happens when gpu image scaling is enabled, which is OFF by default so product and trunk aren't affected.
Assignee: nobody → dzbarsky
Attached patch Patch (obsolete) — Splinter Review
Attachment #625440 - Flags: review?(gal)
Comment on attachment 625440 [details] [diff] [review] Patch I would pull this into a local and then use that further down, but up to you.
Attachment #625440 - Flags: review?(gal) → review+
e/contenteditable.css ../../../layout/style/designmode.css ../../../layout/style/TopLevelImageDocument.css ../../../layout/style/TopLevelVideoDocument.css ../../dist/bin/res /Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp: In member function ‘bool nsDisplayBackground::TryOptimizeToImageLayer(nsDisplayListBuilder*)’: /Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp:1168: error: no match for ‘operator!’ in ‘!state.nsBackgroundLayerState::mImageRenderer. nsImageRenderer::GetContainer()’ /Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp:1168: note: candidates are: operator!(bool) <built-in>
Attached patch patch (obsolete) — Splinter Review
This page often crashes for me without the patch: http://www.mobileasiaexpo.com/ It has a large scaled background images and it loads pretty slowly, causing us to paint before the image is ready, resulting in a crash. Works with my version of the patch. Lets land it.
Yeah, I made the same change locally. I don't have L3 access so I can't actually land anything.
Attached patch PatchSplinter Review
Attachment #625440 - Attachment is obsolete: true
Attachment #625481 - Attachment is obsolete: true
Attachment #625509 - Flags: review+
I am not of much use these days, but comment 6 I can fix!
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: