Bug 42164

Summary: [v8] Web Inspector: remove debugger code that doesn't use ScriptDebugServer
Product: WebKit Reporter: Yury Semikhatsky <yurys>
Component: Web Inspector (Deprecated)Assignee: Yury Semikhatsky <yurys>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, joepeck, keishi, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch pfeldman: review+

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.