Closed Bug 1340457 Opened 7 years ago Closed 7 years ago

Perma failure leakcheck | default process: 120 bytes leaked (CSSVariableValues, nsStyleVariables, nsTArray_base)

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox52 --- unaffected
firefox53 --- unaffected
firefox54 --- fixed

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: intermittent-failure, memory-leak, Whiteboard: [stockwell fixed]) 

this seems a new perma failure
Keywords: mlk
Grr, another half-written comment, I need to start looking at the textbox when switching tabs back. The only nsStyleVariables we should have is the static one we use to stub them, so this is slightly scary.
Well, you are only leaking one. Maybe it is stored in a static variable somewhere, and that isn't being cleaned up at shutdown?
The leak comes from test_variables.html (maybe unsurprisingly).  I'll disable the test for now.
Depends on: 1340766
Yes, I'm pretty sure this comes from the lazily created Variables struct:

lazy_static! {
    static ref EMPTY_VARIABLES_STRUCT: nsStyleVariables = {
        unsafe {
            let mut variables: nsStyleVariables = unsafe { zeroed() };
            Gecko_Construct_nsStyleVariables(&mut variables);
            variables
        }
    };
}
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [stockwell fixed]
You need to log in before you can comment on or make changes to this bug.