[webkit-reviews] review granted: [Bug 58652] Web Inspector: Background network events collection - add GUI to Inspector : [Attachment 93653] Minor fix after pfeldman review

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 18 06:37:15 PDT 2011


Yury Semikhatsky <yurys at chromium.org> has granted Sergey Vorobyev
<sergeyvorobyev at google.com>'s request for review:
Bug 58652: Web Inspector: Background network events collection - add GUI to
Inspector
https://bugs.webkit.org/show_bug.cgi?id=58652

Attachment 93653: Minor fix after pfeldman review
https://bugs.webkit.org/attachment.cgi?id=93653&action=review

------- Additional Comments from Yury Semikhatsky <yurys at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=93653&action=review

> LayoutTests/http/tests/inspector/network/network-clear-after-disabled.html:37

> +    function step123(stepNumber) {

name 'step123' looks weird

> LayoutTests/http/tests/inspector/network/network-clear-after-disabled.html:42

> +	   if (stepNumber._description === "1") {

'if' should go on the same line as 'else'.


More information about the webkit-reviews mailing list