RESOLVED FIXED 93130
Web Inspector: add memory instrumentation for CSSValue and its descendants 
https://bugs.webkit.org/show_bug.cgi?id=93130
Summary Web Inspector: add memory instrumentation for CSSValue and its descendants
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
DetailsFormatted DiffDiff
Patch (76.51 KB, patch)
2012-08-06 02:12 PDT, Yury Semikhatsky 		no flags
DetailsFormatted DiffDiff
Patch (76.52 KB, patch)
2012-08-06 02:27 PDT, Yury Semikhatsky 		no flags
DetailsFormatted DiffDiff
Patch (77.41 KB, patch)
2012-08-06 03:05 PDT, Yury Semikhatsky 		no flags
DetailsFormatted DiffDiff
Patch (77.47 KB, patch)
2012-08-06 04:51 PDT, Yury Semikhatsky 		pfeldman: review+
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Yury Semikhatsky
Comment 1 2012-08-03 09:32:53 PDT
Created attachment 156398 [details] Patch
Early Warning System Bot
Comment 2 2012-08-03 10:04:49 PDT
Comment on attachment 156398 [details] Patch Attachment 156398 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/13434081
Build Bot
Comment 3 2012-08-03 10:14:43 PDT
Comment on attachment 156398 [details] Patch Attachment 156398 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13423861
Early Warning System Bot
Comment 4 2012-08-03 10:43:50 PDT
Comment on attachment 156398 [details] Patch Attachment 156398 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/13426670
Build Bot
Comment 5 2012-08-03 14:05:03 PDT
Comment on attachment 156398 [details] Patch Attachment 156398 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13428711
Yury Semikhatsky
Comment 6 2012-08-06 02:12:19 PDT
Created attachment 156629 [details] Patch
Yury Semikhatsky
Comment 7 2012-08-06 02:27:25 PDT
Created attachment 156631 [details] Patch
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
Comment on attachment 156631 [details] Patch Attachment 156631 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13438638
Yury Semikhatsky
Comment 10 2012-08-06 03:05:33 PDT
Created attachment 156643 [details] Patch
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
Created attachment 156661 [details] Patch
Ilya Tikhonovsky
Comment 15 2012-08-06 06:25:02 PDT
lgtm
Yury Semikhatsky
Comment 16 2012-08-06 07:10:36 PDT
Committed r124768: <http://trac.webkit.org/changeset/124768>
Note You need to log in before you can comment on or make changes to this bug.