[webkit-reviews] review requested: [Bug 47275] Web Inspector: inspector settings/properties/states management should be extracted into separate class. : [Attachment 70347] [patch] polished but very complex solution :) without InspectorProperty* classes and theirs virtual functions. Just for try bots.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Oct 9 02:24:59 PDT 2010


Ilya Tikhonovsky <loislo at chromium.org> has asked  for review:
Bug 47275: Web Inspector: inspector settings/properties/states management
should be extracted into separate class.
https://bugs.webkit.org/show_bug.cgi?id=47275

Attachment 70347: [patch] polished but very complex solution :) without
InspectorProperty* classes and theirs virtual functions. Just for try bots.
https://bugs.webkit.org/attachment.cgi?id=70347&action=review

------- Additional Comments from Ilya Tikhonovsky <loislo at chromium.org>
has no virtual functions
has strong type checking
has no overhead on get* operations.
debugger friendly :)


More information about the webkit-reviews mailing list