MLK:164 bytes- CSS leak - apprunner startup/shutdown process

VERIFIED FIXED in M16

Status

()

P3
normal
VERIFIED FIXED
20 years ago
10 years ago

People

(Reporter: bruce, Assigned: attinasi)

Tracking

({memory-leak})

Trunk
memory-leak
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: 4/24: requested verification by reporter or assigned engineer; waiting for apprunner document leaks to be fixed)

(Reporter)

Description

20 years ago
Build, pull from March 28, 1999.  Happens in apprunner startup/shutdown process
with only default chrome loading.

MLK: 164 bytes leaked in 2 blocks
  * This memory was allocated from:
        malloc         [rtlib.o]
        __bUiLtIn_nEw  [libgcc.a]
        __builtin_new  [rtlib.o]
        __bUiLtIn_vEc_nEw [libgcc.a]
        __builtin_vec_new [rtlib.o]
        nsString::EnsureCapacityFor(int) [nsString.cpp:314]
        nsString::Append(const unsigned short*,int) [nsString.cpp:965]
        nsString::Append(const nsString&,int) [nsString.cpp:918]
        nsCSSValue::GetStringValue(nsString&)const [nsCSSValue.h:182]
        MapDeclarationInto(nsICSSDeclaration*,nsIStyleContext*,nsIPresContext*)
[nsCSSStyleRule.cpp:1566]
        CSSStyleRuleImpl::MapStyleInto(nsIStyleContext*,nsIPresContext*)
[nsCSSStyleRule.cpp:1145]
        MapStyleRule(nsISupports*,void*) [nsStyleContext.cpp:1288]

nsSupportsArray::EnumerateForwards(int(*)(nsISupports*,void*),void*)const
[nsSupportsArray.cpp:318]
        StyleContextImpl::RemapStyle(nsIPresContext*) [nsStyleContext.cpp:1317]

StyleContextImpl::StyleContextImpl(nsIStyleContext*,nsIAtom*,nsISupportsArray*,n
sIPresContext*) [nsStyleContext.cpp:944]

NS_NewStyleContext(nsIStyleContext**,nsIStyleContext*,nsIAtom*,nsISupportsArray*
,nsIPresContext*) [nsStyleContext.cpp:1514]

StyleSetImpl::GetContext(nsIPresContext*,nsIStyleContext*,nsIAtom*,nsISupportsAr
ray*,int,int&) [nsStyleSet.cpp:453]

StyleSetImpl::ProbePseudoStyleFor(nsIPresContext*,nsIContent*,nsIAtom*,nsIStyleC
ontext*,int) [nsStyleSet.cpp:699]

nsPresContext::ProbePseudoStyleContextFor(nsIContent*,nsIAtom*,nsIStyleContext*,
int,nsIStyleContext**) [nsPresContext.cpp:440]

nsToolboxFrame::RefreshStyleContext(nsIPresContext*,nsIAtom*,nsCOMPtr*,nsIConten
t*,nsIStyleContext*) [nsToolboxFrame.cpp:122]

nsToolboxFrame::ReResolveStyleContext(nsIPresContext*,nsIStyleContext*,int,nsSty
leChangeList*,int*) [nsToolboxFrame.cpp:148]
        nsToolboxFrame::Reflow(nsIPresContext&,nsHTMLReflowMetrics&,const
nsHTMLReflowState&,unsigned int&) [nsToolboxFrame.cpp:269]
        nsLineLayout::ReflowFrame(nsIFrame*,nsIFrame**,unsigned int&)
[nsLineLayout.cpp:841]

nsBlockFrame::ReflowInlineFrame(nsBlockReflowState&,nsLineBox*,nsIFrame*,unsigne
d char*) [nsBlockFrame.cpp:2728]
        nsBlockFrame::ReflowInlineFrames(nsBlockReflowState&,nsLineBox*,int*)
[nsBlockFrame.cpp:2610]
        nsBlockFrame::ReflowLine(nsBlockReflowState&,nsLineBox*,int*)
[nsBlockFrame.cpp:1717]
        nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&)
[nsBlockFrame.cpp:1522]
        nsBlockFrame::Reflow(nsIPresContext&,nsHTMLReflowMetrics&,const
nsHTMLReflowState&,unsigned int&) [nsBlockFrame.cpp:892]
        nsBlockReflowContext::ReflowBlock(nsIFrame*,const
nsRect&,int,int,int,nsMargin&,unsigned int&) [nsBlockReflowContext.cpp:212]
        nsBlockFrame::ReflowBlockFrame(nsBlockReflowState&,nsLineBox*,int*)
[nsBlockFrame.cpp:2382]
  * Block of 82 bytes (2 times); last block at 0x735b68

Updated

20 years ago
Status: NEW → ASSIGNED
Target Milestone: M5

Updated

20 years ago
Target Milestone: M5 → M6

Comment 1

20 years ago
Apprunner is currently leaking the entire document. Deferring until I can even
see these leaks again...

Updated

20 years ago
Whiteboard: suspect leaks have been fixed

Updated

20 years ago
Target Milestone: M6 → M7

Comment 2

20 years ago
It seems that RDF generated documents are *still* leaking entirely, not
suprising that all the style data leaks with them. Deferring again until major
leaks are corrected in apprunner.

Updated

20 years ago
Whiteboard: suspect leaks have been fixed → waiting for apprunner document leaks to be fixed

Updated

20 years ago
Target Milestone: M10 → M11

Comment 3

19 years ago
Pushing off non-beta 1 issues
(Reporter)

Updated

19 years ago
Depends on: 13283

Updated

19 years ago
Summary: MLK: CSS leak → MLK:164 bytes- CSS leak - apprunner startup/shutdown process

Updated

19 years ago
Blocks: 14516

Comment 4

19 years ago
Reassigning peterl's bugs to myself.

Comment 5

19 years ago
Accepting peterl's bugs that have a Target Milestone

Comment 6

19 years ago
Pushing my M15 bugs to M16

Updated

19 years ago
Keywords: mlk

Comment 7

19 years ago
Block-moved memory leaks to attinasi
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 8

19 years ago
I cannot find these leaks in the latest leak logs. Since this bug is so old I am 
marking closed.
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Updated

19 years ago
Keywords: verifyme

Comment 9

19 years ago
Mark or Bruce:

I am unable to verify leaks. Could one of you please marked this as verified? 
Thanks
Whiteboard: waiting for apprunner document leaks to be fixed → 4/24: requested verification by reporter or assigned engineer; waiting for apprunner document leaks to be fixed

Comment 10

19 years ago
Verified per Marc's comments.
Status: RESOLVED → VERIFIED

Updated

10 years ago
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.