Bug 139981 – Web Inspector: pass parameters to inspector instrumentation by reference where possible

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Brian Burg]

Cleanup work.

[Radar WebKit Bug Importer]

<rdar://problem/19353559>

[Brian Burg]

Created attachment 243787 [details]
Patch

[WebKit Commit Bot]

Attachment 243787 [details] did not pass style-queue:


ERROR: Source/WebCore/inspector/InspectorInstrumentation.cpp:46:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 42 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Andreas Kling]

Comment on attachment 243787 [details]
Patch

r=me, this is really neat!

Please fix the EFL & Gtk+ build failures before landing:

../../Source/WebCore/page/EventHandler.cpp: In member function 'bool WebCore::EventHandler::handleTouchEvent(const WebCore::PlatformTouchEvent&)':
../../Source/WebCore/page/EventHandler.cpp:3869:82: error: no matching function for call to 'WebCore::InspectorInstrumentation::handleTouchEvent(WebCore::Frame&, WebCore::Node&)'
             if (node && InspectorInstrumentation::handleTouchEvent(m_frame, *node))
                                                                                  ^
../../Source/WebCore/page/EventHandler.cpp:3869:82: note: candidate is:
In file included from ../../Source/WebCore/page/EventHandler.cpp:63:0:
../../Source/WebCore/inspector/InspectorInstrumentation.h:746:17: note: static bool WebCore::InspectorInstrumentation::handleTouchEvent(WebCore::Page&, WebCore::Node&)
 inline bool InspectorInstrumentation::handleTouchEvent(Page& page, Node& node)
                 ^
../../Source/WebCore/inspector/InspectorInstrumentation.h:746:17: note:   no known conversion for argument 1 from 'WebCore::Frame' to 'WebCore::Page&'

[Brian Burg]

Created attachment 243822 [details]
Fix EFL/GTK

[WebKit Commit Bot]

Attachment 243822 [details] did not pass style-queue:


ERROR: Source/WebCore/inspector/InspectorInstrumentation.cpp:46:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 42 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Brian Burg]

Committed r177925: <http://trac.webkit.org/changeset/177925>

[Csaba Osztrogonác]

(In reply to comment #7)
> Committed r177925: <http://trac.webkit.org/changeset/177925>

It broke the Apple Windows build:
     1>WebKit.exp : error LNK2001: unresolved external symbol "class WebCore::InstrumentingAgents * __cdecl WebCore::instrumentationForPage(class WebCore::Page *)" (?instrumentationForPage@WebCore@@YAPAVInstrumentingAgents@1@PAVPage@1@@Z)
     1>WebKit.exp : error LNK2001: unresolved external symbol "private: static class WebCore::InstrumentingAgents * __cdecl WebCore::InspectorInstrumentation::instrumentingAgentsForPage(class WebCore::Page *)" (?instrumentingAgentsForPage@InspectorInstrumentation@WebCore@@CAPAVInstrumentingAgents@2@PAVPage@2@@Z)
     1>C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\bin32\WebKit.dll : fatal error LNK1120: 2 unresolved externals
     1>Done Building Project "C:\cygwin\home\buildbot\slave\win-release\build\Source\WebKit\WebKit.vcxproj\WebKit\WebKit.vcxproj" (Build target(s)) -- FAILED.

[Brent Fulgham]

A small build fix for Windows was needed:

http://trac.webkit.org/changeset/177940

Looks like you also found a crufty symbol export we don't need anymore! :-)