[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