I'd like to upstream core memory instrumentation code to WTF. This will give us a chance to implement reportMemoryUsage members for containers. On this way I have to remove WebCore specific members from MemoryInstrumentation class.
Created attachment 162200 [details] Patch
Comment on attachment 162200 [details] Patch Attachment 162200 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13765140
Created attachment 162219 [details] patch for mac try bot
(In reply to comment #2) > (From update of attachment 162200 [details]) > Attachment 162200 [details] did not pass mac-ews (mac): > Output: http://queues.webkit.org/results/13765140 looks like gcc bug http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15867
Comment on attachment 162219 [details] patch for mac try bot Attachment 162219 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13741894
Created attachment 162225 [details] another patch for mac try bot
Comment on attachment 162225 [details] another patch for mac try bot View in context: https://bugs.webkit.org/attachment.cgi?id=162225&action=review > Source/WebCore/dom/MemoryInstrumentation.cpp:55 > +template <> void MemoryInstrumentationTraits::addInstrumentedObject<String>(MemoryInstrumentation* instrumentation, const String* const& string, MemoryObjectType ownerObjectType, MemoryOwningType owningType) template <> -> template<> here and in other places. > Source/WebCore/dom/MemoryInstrumentation.h:54 > +class MemoryInstrumentationTraits { Please add an explanation(at least in the ChangeLog) why we cannot use template class here.
Created attachment 162241 [details] Patch
Comment on attachment 162241 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=162241&action=review > Source/WebCore/dom/MemoryInstrumentation.h:393 > +template<> void MemoryInstrumentationTraits::addInstrumentedObject<const KURL>(MemoryInstrumentation*, const KURL* const&, MemoryObjectType, MemoryOwningType); Why do we need overload for T in addition to const T ?
Committed r127592: <http://trac.webkit.org/changeset/127592>