Closed Bug 1251726 Opened 9 years ago Closed 9 years ago

Intermittent test_bug879717.html,test_streams_element_capture_reset.html,test_streams_autoplay.html | application crashed [@ mozilla::layers::BufferTextureHost::PrepareTextureSource] or application crashed [None]

Categories

(Core :: Graphics: Layers, defect)

Product:
Core
Component:
Graphics: Layers
Version:
47 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox47 --- fixed
firefox48 --- fixed

People

(Reporter: KWierso, Assigned: sotaro)

References

Details

(Keywords: intermittent-failure, Whiteboard: [gfx-noted])

Attachments

(1 file, 1 obsolete file)

patch - Check if Compositor is set
(deleted), patch
nical
: review+
ritu
: approval-mozilla-aurora+
Details | Diff | Splinter Review
Looks like a gfx bug.
Component: Audio/Video → Graphics: Layers
Nical, in the second report, looks like it is crashing here: https://dxr.mozilla.org/mozilla-central/source/gfx/layers/composite/TextureHost.cpp#546 mCompositor seems null there at the time PrepareTextureSource is called. Seems regression happened with: https://hg.mozilla.org/mozilla-central/rev/eced4790cbdc
Flags: needinfo?(nical.bugzilla)
Whiteboard: [gfx-noted]
Comment on attachment 8726690 [details] [diff] [review] patch - Make sure and check if Compositor is set. milan, can you review the patch?
Attachment #8726690 - Flags: review?(milan)
Attachment #8726690 - Flags: review?(milan)
Attachment #8726690 - Attachment is obsolete: true
Attachment #8726723 - Flags: review?(nical.bugzilla)
Summary: Intermittent test_streams_element_capture_reset.html | application crashed [@ mozilla::layers::BufferTextureHost::PrepareTextureSource] or application crashed [None] → Intermittent test_bug879717.html,test_streams_element_capture_reset.html | application crashed [@ mozilla::layers::BufferTextureHost::PrepareTextureSource] or application crashed [None]
Attachment #8726723 - Flags: review?(nical.bugzilla) → review+
Summary: Intermittent test_bug879717.html,test_streams_element_capture_reset.html | application crashed [@ mozilla::layers::BufferTextureHost::PrepareTextureSource] or application crashed [None] → Intermittent test_bug879717.html,test_streams_element_capture_reset.html,test_streams_autoplay.html | application crashed [@ mozilla::layers::BufferTextureHost::PrepareTextureSource] or application crashed [None]
Assignee: nobody → sotaro.ikeda.g
Flags: needinfo?(nical.bugzilla)
https://hg.mozilla.org/mozilla-central/rev/185e10fbb9d3
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Please nominate this for Aurora approval when you get a chance.
Flags: needinfo?(sotaro.ikeda.g)
Blocks: 1249273
Flags: needinfo?(sotaro.ikeda.g)
Comment on attachment 8726723 [details] [diff] [review] patch - Check if Compositor is set Approval Request Comment [Feature/regressing bug #]: Bug 1249273 [User impact if declined]: It could cause the crash during starting firefox. [Describe test coverage new/current, TreeHerder]: landed to m-c. [Risks and why]: low. It just added pointer check. [String/UUID change made/needed]:none
Attachment #8726723 - Flags: approval-mozilla-aurora?
Comment on attachment 8726723 [details] [diff] [review] patch - Check if Compositor is set fixes an intermittent, taking it.
Attachment #8726723 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: