Bug 93130

Summary: Web Inspector: add memory instrumentation for CSSValue and its descendants
Product: WebKit Reporter: Yury Semikhatsky <yurys>
Component: Web Inspector (Deprecated)Assignee: Yury Semikhatsky <yurys>
Status: RESOLVED FIXED    
Severity: Normal CC: alph, apavlov, bweinstein, cmarcelo, eric.carlson, feature-media-reviews, gustavo, joepeck, keishi, koivisto, loislo, macpherson, menard, pfeldman, philn, pmuellr, rik, timothy, webkit.review.bot, xan.lopez, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 92748    
Bug Blocks: 87262    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch pfeldman: review+

Yury Semikhatsky
Reported 2012-08-03 09:21:18 PDT
It is the next step required for reporting memory footprint for CSSStyleSheet and related objects.
Attachments
Patch (68.24 KB, patch)
2012-08-03 09:32 PDT, Yury Semikhatsky
no flags
Patch (76.51 KB, patch)
2012-08-06 02:12 PDT, Yury Semikhatsky
no flags
Patch (76.52 KB, patch)
2012-08-06 02:27 PDT, Yury Semikhatsky
no flags
Patch (77.41 KB, patch)
2012-08-06 03:05 PDT, Yury Semikhatsky
no flags
Patch (77.47 KB, patch)
2012-08-06 04:51 PDT, Yury Semikhatsky
pfeldman: review+
Yury Semikhatsky
Comment 1 2012-08-03 09:32:53 PDT
Early Warning System Bot
Comment 2 2012-08-03 10:04:49 PDT
Build Bot
Comment 3 2012-08-03 10:14:43 PDT
Early Warning System Bot
Comment 4 2012-08-03 10:43:50 PDT
Build Bot
Comment 5 2012-08-03 14:05:03 PDT
Yury Semikhatsky
Comment 6 2012-08-06 02:12:19 PDT
Yury Semikhatsky
Comment 7 2012-08-06 02:27:25 PDT
Ilya Tikhonovsky
Comment 8 2012-08-06 02:44:42 PDT
Comment on attachment 156631 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=156631&action=review > Source/WebCore/css/CSSCrossfadeValue.cpp:147 > + MemoryClassInfo<CSSCrossfadeValue> info(memoryObjectInfo, this, MemoryInstrumentation::CSS); > + info.addInstrumentedMember(m_fromValue); > + info.addInstrumentedMember(m_toValue); > + info.addInstrumentedMember(m_percentageValue); > + // FIXME: add instrumentation for > + // m_cachedFromImage > + // m_cachedToImage > + // m_generatedImage looks like you missed to call CSSImageGeneratorValue::reportDescendantMemoryUsage
Build Bot
Comment 9 2012-08-06 02:59:11 PDT
Yury Semikhatsky
Comment 10 2012-08-06 03:05:33 PDT
Yury Semikhatsky
Comment 11 2012-08-06 03:06:14 PDT
(In reply to comment #8) > (From update of attachment 156631 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=156631&action=review > > > Source/WebCore/css/CSSCrossfadeValue.cpp:147 > > + MemoryClassInfo<CSSCrossfadeValue> info(memoryObjectInfo, this, MemoryInstrumentation::CSS); > > + info.addInstrumentedMember(m_fromValue); > > + info.addInstrumentedMember(m_toValue); > > + info.addInstrumentedMember(m_percentageValue); > > + // FIXME: add instrumentation for > > + // m_cachedFromImage > > + // m_cachedToImage > > + // m_generatedImage > > looks like you missed to call CSSImageGeneratorValue::reportDescendantMemoryUsage Added CSSImageGeneratorValue::reportBaseClassMemoryUsage call
Ilya Tikhonovsky
Comment 12 2012-08-06 04:30:13 PDT
Comment on attachment 156643 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=156643&action=review > Source/WebCore/css/CSSCursorImageValue.cpp:139 > + MemoryClassInfo<CSSCursorImageValue> info(memoryObjectInfo, this, MemoryInstrumentation::CSS); > +#if ENABLE(SVG) CSSImageValue::reportBaseClassMemoryUsage ?
Yury Semikhatsky
Comment 13 2012-08-06 04:37:29 PDT
(In reply to comment #11) > (In reply to comment #8) > > (From update of attachment 156631 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=156631&action=review > > > > > Source/WebCore/css/CSSCrossfadeValue.cpp:147 > > > + MemoryClassInfo<CSSCrossfadeValue> info(memoryObjectInfo, this, MemoryInstrumentation::CSS); > > > + info.addInstrumentedMember(m_fromValue); > > > + info.addInstrumentedMember(m_toValue); > > > + info.addInstrumentedMember(m_percentageValue); > > > + // FIXME: add instrumentation for > > > + // m_cachedFromImage > > > + // m_cachedToImage > > > + // m_generatedImage > > > > looks like you missed to call CSSImageGeneratorValue::reportDescendantMemoryUsage > > Added CSSImageGeneratorValue::reportBaseClassMemoryUsage call Done.
Yury Semikhatsky
Comment 14 2012-08-06 04:51:17 PDT
Ilya Tikhonovsky
Comment 15 2012-08-06 06:25:02 PDT
lgtm
Yury Semikhatsky
Comment 16 2012-08-06 07:10:36 PDT
Note You need to log in before you can comment on or make changes to this bug.