Bug 96511 – Web Inspector: NMI: migrate core instrumentation code to WTF namespace

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Ilya Tikhonovsky]

The only problem is a custom trait for KURL that looks weird if instrumentation code is moved to WTF
I'll instrument it via normal reportMemoryUsage in the next patch.

[Ilya Tikhonovsky]

Created attachment 163623 [details]
Patch

[Gyuyoung Kim]

Comment on attachment 163623 [details]
Patch

Attachment 163623 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/13824725

[Early Warning System Bot]

Comment on attachment 163623 [details]
Patch

Attachment 163623 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/13833405

[Early Warning System Bot]

Comment on attachment 163623 [details]
Patch

Attachment 163623 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/13835229

[Ilya Tikhonovsky]

Created attachment 163634 [details]
Patch

[Early Warning System Bot]

Comment on attachment 163634 [details]
Patch

Attachment 163634 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/13821837

[Ilya Tikhonovsky]

(In reply to comment #6)
> (From update of attachment 163634 [details])
> Attachment 163634 [details] did not pass qt-wk2-ews (qt):
> Output: http://queues.webkit.org/results/13821837

Looks like problem with dependency tracking.

[Yury Semikhatsky]

Comment on attachment 163634 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=163634&action=review

> Source/WebCore/bindings/js/ScriptProfiler.cpp:41
> +#include <wtf/Forward.h>

You already included it in the header file.

> Source/WebCore/css/MediaQuery.h:-39
> -class MemoryObjectInfo;

How does this class declaration come here now?

[Ilya Tikhonovsky]

(In reply to comment #8)
> (From update of attachment 163634 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=163634&action=review
> 
> > Source/WebCore/bindings/js/ScriptProfiler.cpp:41
> > +#include <wtf/Forward.h>
> 
> You already included it in the header file.
> 
> > Source/WebCore/css/MediaQuery.h:-39
> > -class MemoryObjectInfo;
> 
> How does this class declaration come here now?

It comes from wtf/Forward.h

[Build Bot]

Comment on attachment 163634 [details]
Patch

Attachment 163634 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/13839196

[Ilya Tikhonovsky]

Created attachment 163776 [details]
Patch

[Ilya Tikhonovsky]

Created attachment 163784 [details]
with fix for mac build

[Early Warning System Bot]

Comment on attachment 163784 [details]
with fix for mac build

Attachment 163784 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/13824984

[Yury Semikhatsky]

(In reply to comment #9)
> (In reply to comment #8)
> > (From update of attachment 163634 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=163634&action=review
> > 
> > > Source/WebCore/bindings/js/ScriptProfiler.cpp:41
> > > +#include <wtf/Forward.h>
> > 
> > You already included it in the header file.
> > 
> > > Source/WebCore/css/MediaQuery.h:-39
> > > -class MemoryObjectInfo;
> > 
> > How does this class declaration come here now?
> 
> It comes from wtf/Forward.h

Still don't see it.

[Ilya Tikhonovsky]

(In reply to comment #14)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > (From update of attachment 163634 [details] [details] [details])
> > > View in context: https://bugs.webkit.org/attachment.cgi?id=163634&action=review
> > > 
> > > > Source/WebCore/bindings/js/ScriptProfiler.cpp:41
> > > > +#include <wtf/Forward.h>
> > > 
> > > You already included it in the header file.
> > > 
> > > > Source/WebCore/css/MediaQuery.h:-39
> > > > -class MemoryObjectInfo;
> > > 
> > > How does this class declaration come here now?
> > 
> > It comes from wtf/Forward.h
> 
> Still don't see it.

I found that it is including indirectly via 
wtf/WTFString.h -> wtf/StringImpl.h -> wtf/Forward.h

[Ilya Tikhonovsky]

(In reply to comment #15)
> (In reply to comment #14)
> > (In reply to comment #9)
> > > (In reply to comment #8)
> > > > (From update of attachment 163634 [details] [details] [details] [details])
> > > > View in context: https://bugs.webkit.org/attachment.cgi?id=163634&action=review
> > > > 
> > > > > Source/WebCore/bindings/js/ScriptProfiler.cpp:41
> > > > > +#include <wtf/Forward.h>
> > > > 
> > > > You already included it in the header file.
> > > > 
> > > > > Source/WebCore/css/MediaQuery.h:-39
> > > > > -class MemoryObjectInfo;
> > > > 
> > > > How does this class declaration come here now?
> > > 
> > > It comes from wtf/Forward.h
> > 
> > Still don't see it.
> 
> I found that it is including indirectly via 
> wtf/WTFString.h -> wtf/StringImpl.h -> wtf/Forward.h

I think I can remove this include from StringImpl.h
Looks like it is using only for "template <typename T> class StringBuffer;"
But this change will affect toщ much cpp files so I'd like to do this in a separate patch.

[Ilya Tikhonovsky]

Committed r128418: <http://trac.webkit.org/changeset/128418>