RESOLVED FIXED 107278
Web Inspector: Native Memory Instrumentation: replace nodeName argument with className 
https://bugs.webkit.org/show_bug.cgi?id=107278
Summary Web Inspector: Native Memory Instrumentation: replace nodeName argument with ...
Ilya Tikhonovsky
Reported 2013-01-18 06:51:45 PST
I found that in all the cases we need className when we report raw and private buffers and never need nodeName.
Attachments
Patch (15.53 KB, patch)
2013-01-18 06:55 PST, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
Patch (5.47 KB, patch)
2013-01-18 10:28 PST, Ilya Tikhonovsky 		yurys: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2013-01-18 06:55:08 PST
Created attachment 183444 [details] Patch
Yury Semikhatsky
Comment 2 2013-01-18 07:23:37 PST
Comment on attachment 183444 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183444&action=review > Source/WTF/wtf/MemoryInstrumentation.h:261 > + MemoryClassInfo info(memoryObjectInfo, object, 0, sizeof(T)); How is it related to this change? > Source/WTF/wtf/MemoryInstrumentationHashCountedSet.h:45 > + info.addPrivateBuffer(sizeof(typename HashMapType::ValueType) * hashSet->capacity(), "ValueType[]", "data"); It seems that we can derive actual name for ValueType automatically the same way as you do for other classes in https://bugs.webkit.org/show_bug.cgi?id=107262. > Source/WTF/wtf/MemoryInstrumentationParsedURL.h:43 > + info.addMember(urlString->string(), "string"); This seems unrelated. You can do this as part of https://bugs.webkit.org/show_bug.cgi?id=107262 or similar bug.
Ilya Tikhonovsky
Comment 3 2013-01-18 10:28:26 PST
Created attachment 183499 [details] Patch
Ilya Tikhonovsky
Comment 4 2013-01-31 22:53:57 PST
Committed r141539: <http://trac.webkit.org/changeset/141539>
Note You need to log in before you can comment on or make changes to this bug.