[webkit-reviews] review granted: [Bug 35148] Web Inspector: support debugging of workers. : [Attachment 49388] A patch to enable early injection of script into inspected page and a fake worker implementation to be injected when debugging workers is enabled (review suggestions addressed)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Feb 24 07:55:23 PST 2010


Pavel Feldman <pfeldman at chromium.org> has granted Andrey Kosyakov
<caseq at chromium.org>'s request for review:
Bug 35148: Web Inspector: support debugging of workers.
https://bugs.webkit.org/show_bug.cgi?id=35148

Attachment 49388: A patch to enable early injection of script into inspected
page and a fake worker implementation to be injected when debugging workers is
enabled (review suggestions addressed)
https://bugs.webkit.org/attachment.cgi?id=49388&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
A bunch of style nits, otherwise looks good. Please go through the { }
alignment, blank lines and wrapped lines, then we can do cq+!
Btw, where is the injecting code itself?


More information about the webkit-reviews mailing list