RESOLVED FIXED 42164
[v8] Web Inspector: remove debugger code that doesn't use ScriptDebugServer 
https://bugs.webkit.org/show_bug.cgi?id=42164
Summary [v8] Web Inspector: remove debugger code that doesn't use ScriptDebugServer
Yury Semikhatsky
Reported 2010-07-13 07:07:40 PDT
There is some v8-specific code that is obsoleted by implementation of ScriptDebugServer for v8. It should be removed.
Attachments
Patch (24.65 KB, patch)
2010-07-14 01:26 PDT, Yury Semikhatsky 		no flags
DetailsFormatted DiffDiff
Patch (79.42 KB, patch)
2010-07-14 02:58 PDT, Yury Semikhatsky 		pfeldman: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Yury Semikhatsky
Comment 1 2010-07-14 01:26:05 PDT
Created attachment 61482 [details] Patch
Yury Semikhatsky
Comment 2 2010-07-14 02:58:49 PDT
Created attachment 61497 [details] Patch
Yury Semikhatsky
Comment 3 2010-07-14 05:18:16 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ... D WebKit/chromium/src/js/DebuggerAgent.js M WebCore/ChangeLog M WebCore/bindings/v8/ScriptDebugServer.cpp M WebCore/bindings/v8/ScriptDebugServer.h M WebKit/chromium/ChangeLog M WebKit/chromium/WebKit.gypi M WebKit/chromium/src/DebuggerAgentManager.cpp M WebKit/chromium/src/InspectorFrontendClientImpl.cpp M WebKit/chromium/src/WebDevToolsAgentImpl.cpp M WebKit/chromium/src/js/DevTools.js M WebKit/chromium/src/js/DevToolsHostStub.js M WebKit/chromium/src/js/InspectorControllerImpl.js M WebKit/chromium/src/js/Tests.js Committed r63311
Note You need to log in before you can comment on or make changes to this bug.