Closed Bug 4345 Opened 25 years ago Closed 25 years ago

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

Categories

(Core :: CSS Parsing and Computation, defect, P3)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: attinasi)

References

Details

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

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
Status: NEW → ASSIGNED
Target Milestone: M5
Target Milestone: M5 → M6
Apprunner is currently leaking the entire document. Deferring until I can even
see these leaks again...
Whiteboard: suspect leaks have been fixed
Target Milestone: M6 → M7
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.
Whiteboard: suspect leaks have been fixed → waiting for apprunner document leaks to be fixed
Target Milestone: M10 → M11
Pushing off non-beta 1 issues
Depends on: 13283
Summary: MLK: CSS leak → MLK:164 bytes- CSS leak - apprunner startup/shutdown process
Blocks: 14516
Reassigning peterl's bugs to myself.
Accepting peterl's bugs that have a Target Milestone
Pushing my M15 bugs to M16
Keywords: mlk
Block-moved memory leaks to attinasi
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
I cannot find these leaks in the latest leak logs. Since this bug is so old I am 
marking closed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Keywords: verifyme
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
Verified per Marc's comments.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.