[webkit-reviews] review requested: [Bug 27541] Web Inspector: Split InspectorController into InspectorController and InspectorBackend : [Attachment 33333] Step 1

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 23 08:52:03 PDT 2009


Pavel Feldman <pfeldman at chromium.org> has asked  for review:
Bug 27541: Web Inspector: Split InspectorController into InspectorController
and InspectorBackend
https://bugs.webkit.org/show_bug.cgi?id=27541

Attachment 33333: Step 1
https://bugs.webkit.org/attachment.cgi?id=33333&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
Step 1: 
- split classes, 
- move whatever is trivial to the InspectorBackend class
- hide private stuff in InspectorController
- delegate non-trivial calls such as highlighting-related.


More information about the webkit-reviews mailing list