The protocol domain name was changed a few days ago. It'd be nice to change corresponding backend agent name accordingly.
Created attachment 92393 [details] [patch] initial version
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
Created attachment 92402 [details] [patch] second version
manually landed as r85850