[webkit-reviews] review requested: [Bug 40134] Web Inspector: sendMessageToFrontend implementation required. : [Attachment 57915] [patch] third iteration

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jun 4 14:30:36 PDT 2010


Ilya Tikhonovsky <loislo at chromium.org> has asked  for review:
Bug 40134: Web Inspector: sendMessageToFrontend implementation required.
https://bugs.webkit.org/show_bug.cgi?id=40134

Attachment 57915: [patch] third iteration
https://bugs.webkit.org/attachment.cgi?id=57915&action=review

------- Additional Comments from Ilya Tikhonovsky <loislo at chromium.org>
Safari specific code for sendMessage was moved to new class
InspectorClientLocal.
We have attachFrontend/disconnectFrontend pair and that was inconsistent.
Now it is consistent and I think there is no principal difference between
connect/disconnect and attach/detach pairs.
I can use any.
Qt implementation was added.


More information about the webkit-reviews mailing list