stylo: Intermittent leakcheck | default process: 16 bytes leaked (nsStringBuffer)

RESOLVED FIXED in Firefox 55

Status

()

P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: intermittent-bug-filer, Assigned: xidorn)

Tracking

({intermittent-failure, memory-leak})

unspecified
mozilla55
intermittent-failure, memory-leak
Points:
---

Firefox Tracking Flags

(firefox52 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55 fixed)

Details

(Whiteboard: [stockwell disabled])

Component: General → General
Product: Chat Core → Core
Version: trunk → unspecified
Keywords: mlk
bobby: this seems a new intermittent leak in stylo tests, could you take a look, thanks!
Flags: needinfo?(bobbyholley)
Looks like this started with the landing of bug 1315601. Seems to be happening about 80% of the time so we should jump on this quickly.

Over to xidorn.
Assignee: nobody → xidorn+moz
Blocks: 1315601
Flags: needinfo?(bobbyholley) → needinfo?(xidorn+moz)
Priority: -- → P1
Summary: Intermittent leakcheck | default process: 16 bytes leaked (nsStringBuffer) → stylo: Intermittent leakcheck | default process: 16 bytes leaked (nsStringBuffer)
Component: General → CSS Parsing and Computation
(Assignee)

Comment 3

2 years ago
The earliest push with this leak is from bug 1341690: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=8949490f91a7c560485e1d5a350718924b4cb9a2&selectedJob=83324109

The URL change seems to be a more sensible candidate since bug 1315601 doesn't have much work on string stuff.

I assume heycam would know more.
Flags: needinfo?(xidorn+moz) → needinfo?(cam)
(Assignee)

Comment 4

2 years ago
FWIW, according to my local run last night, it seems to me the leak happens inside one of layout/reftests/{dom,generated-content}. I saw them intermittently, but failed to further locate where the issue is.
Also, this bug can be from bug 1296477, which makes more sense with the generated-content bits. manish?
Flags: needinfo?(manishearth)
(Assignee)

Comment 6

2 years ago
Yeah, that is possible as well, since the task crashes before bug 1341690's push.
(Assignee)

Updated

2 years ago
No longer blocks: 1315601
Comment hidden (Intermittent Failures Robot)
(Assignee)

Comment 8

2 years ago
It is layout/reftests/dom/xbl-children-3.xhtml leaking.

I'm going to disable this test in this bug, and file a new bug investigating the reason.
Flags: needinfo?(manishearth)
Flags: needinfo?(cam)
(Assignee)

Comment 10

2 years ago
Filed bug 1347065.
thanks for disabling this test
Whiteboard: [stockwell disabled]

Comment 12

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ce635599e0c1
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
status-firefox52: --- → unaffected
status-firefox53: --- → unaffected
status-firefox54: --- → unaffected
status-firefox-esr52: --- → unaffected
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
You need to log in before you can comment on or make changes to this bug.