Summary: | Web Inspector: Native Memory Instrumentation: MemoryInstrumentation doesn't detect reportMemoryUsage method defined in a base class. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ilya Tikhonovsky <loislo> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Ilya Tikhonovsky <loislo> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | apavlov, benjamin, keishi, loislo, ojan.autocc, pfeldman, pmuellr, vsevik, web-inspector-bugs, webkit.review.bot, yurys | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Ilya Tikhonovsky
2012-12-14 07:50:44 PST
Created attachment 179485 [details]
Patch
The coverage increases by ~3% on cnn, nytimes and google. Comment on attachment 179485 [details] Patch Attachment 179485 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/15310752 Comment on attachment 179485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179485&action=review > Source/WTF/wtf/MemoryInstrumentation.h:133 > + struct Base : public Type, public BaseMixin { }; Will it compile if there is no default constructor for Type ? Created attachment 179698 [details]
for try-bots. speculative fix for windows
Comment on attachment 179485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179485&action=review >> Source/WTF/wtf/MemoryInstrumentation.h:133 >> + struct Base : public Type, public BaseMixin { }; > > Will it compile if there is no default constructor for Type ? Why is it called Base, shouldn't it be Derived instead? > Source/WTF/wtf/MemoryInstrumentation.h:144 > + template <int> template <bool> ? Committed r137892: <http://trac.webkit.org/changeset/137892> Reverted r137892 for reason: it broke compilation on windows Committed r137893: <http://trac.webkit.org/changeset/137893> Committed r137911: <http://trac.webkit.org/changeset/137911> |