RESOLVED FIXED 60256
Web Inspector: rename BrowserDebugger to DOMDebugger 
https://bugs.webkit.org/show_bug.cgi?id=60256
Summary Web Inspector: rename BrowserDebugger to DOMDebugger
Ilya Tikhonovsky
Reported 2011-05-05 01:18:14 PDT
The protocol domain name was changed a few days ago. It'd be nice to change corresponding backend agent name accordingly.
Attachments
[patch] initial version (68.38 KB, patch)
2011-05-05 02:13 PDT, Ilya Tikhonovsky 		no flags
DetailsFormatted DiffDiff
[patch] second version (68.33 KB, patch)
2011-05-05 04:34 PDT, Ilya Tikhonovsky 		yurys: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2011-05-05 02:13:15 PDT
Created attachment 92393 [details] [patch] initial version
Yury Semikhatsky
Comment 2 2011-05-05 04:01:28 PDT
Comment on attachment 92393 [details] [patch] initial version View in context: https://bugs.webkit.org/attachment.cgi?id=92393&action=review Looks good except that DOMDebuggerAgent variables should become domDebuggerAgent > Source/WebCore/ChangeLog:9 > + browserDebugger => DOMDebugger should be domDebugger, not DOMDebugger > Source/WebCore/inspector/CodeGeneratorInspector.pm:41 > + "domainAccessor" => "m_DOMDebuggerAgent", m_domDebuggerAgent > Source/WebCore/inspector/InspectorAgent.h:136 > + InspectorDOMDebuggerAgent* DOMDebuggerAgent() const { return m_DOMDebuggerAgent.get(); } domDebuggerAgent (see domStorageAgent above) > Source/WebCore/inspector/InspectorAgent.h:223 > + OwnPtr<InspectorDOMDebuggerAgent> m_DOMDebuggerAgent; ditto > Source/WebCore/inspector/InspectorInstrumentation.cpp:147 > + DOMDebuggerAgent->willModifyDOMAttr(element); DOMDebuggerAgent -> domDebuggerAgent
Ilya Tikhonovsky
Comment 3 2011-05-05 04:34:16 PDT
Created attachment 92402 [details] [patch] second version
Ilya Tikhonovsky
Comment 4 2011-05-05 22:27:28 PDT
manually landed as r85850
Note You need to log in before you can comment on or make changes to this bug.