Closed
Bug 1514528
Opened 7 years ago
Closed 7 years ago
Crash in MergeState::ProcessOldNode
Categories
(Core :: Graphics: Layers, defect, P1)
Core
Graphics: Layers
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr60 | --- | unaffected |
| firefox64 | --- | unaffected |
| firefox65 | --- | unaffected |
| firefox66 | blocking | fixed |
People
(Reporter: kbrosnan, Assigned: mattwoodrow)
References
Details
(Keywords: crash, regression, topcrash)
Crash Data
Attachments
(1 file)
This bug was filed from the Socorro interface and is
report bp-2e9b00d1-457a-47c6-870d-a92880181215.
=============================================================
Top 10 frames of crashing thread:
0 xul.dll void MergeState::ProcessOldNode layout/painting/RetainedDisplayListBuilder.cpp:525
1 xul.dll class RetainedDisplayList MergeState::Finalize layout/painting/RetainedDisplayListBuilder.cpp:461
2 xul.dll bool RetainedDisplayListBuilder::MergeDisplayLists layout/painting/RetainedDisplayListBuilder.cpp:669
3 xul.dll class mozilla::Maybe<Index<MergedListUnits> > MergeState::ProcessItemFromNewList layout/painting/RetainedDisplayListBuilder.cpp:363
4 xul.dll bool RetainedDisplayListBuilder::MergeDisplayLists layout/painting/RetainedDisplayListBuilder.cpp:666
5 xul.dll class mozilla::Maybe<Index<MergedListUnits> > MergeState::ProcessItemFromNewList layout/painting/RetainedDisplayListBuilder.cpp:363
6 xul.dll bool RetainedDisplayListBuilder::MergeDisplayLists layout/painting/RetainedDisplayListBuilder.cpp:666
7 xul.dll RetainedDisplayListBuilder::AttemptPartialUpdate layout/painting/RetainedDisplayListBuilder.cpp:1336
8 xul.dll nsLayoutUtils::PaintFrame layout/base/nsLayoutUtils.cpp:3620
9 xul.dll mozilla::PresShell::Paint layout/base/PresShell.cpp:6011
=============================================================
Crashed on startup restoring video playback from https://onlinexperiences.com/scripts/Server.nxp
Comment 1•7 years ago
|
||
Got it on https://www.moulinex.fr/Petit-D%C3%A9jeuner/Machines-%C3%A0-pain/Machine-%C3%A0-pain/PAIN-PLAISIR-1KG/p/7211002269 while "playing" with the pictures. (I can't reproduce when trying again)
=> https://crash-stats.mozilla.org/report/index/7df47ac3-758d-4a15-aeee-307780181216
Comment 2•7 years ago
|
||
I always crash when I try to load https://www.mydealz.de/new (bp-e261dec4-cbe8-426f-b701-cace50181216)
Comment 3•7 years ago
|
||
Seeing this on many websites, makes nightly near unusable.
This is my about:support for crash https://crash-stats.mozilla.org/report/index/bp-4f2221ad-e23e-400e-8855-972520181216 in Firefox Nightly on my Windows 10 (64-bit) system.
I got the crash when doing a search for Broadway Gondolier on page https://dvd.netflix.com/Movie/Hover/81001119 (NOT reproducible).
Comment 5•7 years ago
|
||
I tried to find the regression with mozregression but the crash is not 100% reproduce able for me.
I'm not certain that the result is correct.
This is the result https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d86d184dc7d6aa246ada69063e07dbb503c33092&tochange=10aa1d02448438169fc49d467dd7f9a0fd70ca1f which contains the fix for bug 1500864
Flags: needinfo?(matt.woodrow)
Updated•7 years ago
|
Blocks: 1500864
status-firefox65:
--- → unaffected
No longer depends on: 1500864
Keywords: regression,
topcrash
OS: Windows 10 → All
Hardware: x86_64 → All
Comment 6•7 years ago
|
||
(In reply to Matthias Versen [:Matti] from comment #5)
> I tried to find the regression with mozregression but the crash is not 100%
> reproduce able for me.
>
> I'm not certain that the result is correct.
> This is the result
> https://hg.mozilla.org/mozilla-central/
> pushloghtml?fromchange=d86d184dc7d6aa246ada69063e07dbb503c33092&tochange=10aa
> 1d02448438169fc49d467dd7f9a0fd70ca1f which contains the fix for bug 1500864
The code at the top of all of my crashes come from that bug, so I think it's a pretty good bet.
Comment 7•7 years ago
|
||
Let's run with that. Backed out bug 1500864 and triggered new nightlies.
https://hg.mozilla.org/mozilla-central/rev/36a2f27ecc47
Assignee: nobody → matt.woodrow
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox64:
--- → unaffected
status-firefox-esr60:
--- → unaffected
tracking-firefox66:
--- → blocking
Flags: needinfo?(matt.woodrow)
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Updated•7 years ago
|
Flags: qe-verify+
Whiteboard: [qa-triaged]
Updated•7 years ago
|
QA Whiteboard: [qa-triaged]
Whiteboard: [qa-triaged]
Comment 10•6 years ago
|
||
Considering that the initial page to test this issue is "Temporarily unavailable", is there another way I could verify it?
Flags: needinfo?(matt.woodrow)
| Assignee | ||
Comment 11•6 years ago
|
||
The regressing change got backed out, so I don't think you need to worry about verifying if the original link doesn't work.
Flags: needinfo?(matt.woodrow)
Comment 12•6 years ago
|
||
Considering the comment above, I will remove the "qe-verify+" tag. Thank you.
Flags: qe-verify+
Comment 13•5 years ago
|
||
Please specify a root cause for this bug. See :tmaity for more information.
Root Cause: --- → ?
| Assignee | ||
Updated•5 years ago
|
Root Cause: ? → Coding: Logical Error
You need to log in
before you can comment on or make changes to this bug.
Description
•