[webkit-reviews] review denied: [Bug 106705] Web Inspector: add text editor formatter test : [Attachment 182582] Patch

[bugzilla-daemon at webkit.org]

Pavel Feldman <pfeldman at chromium.org> has denied Andrey Lushnikov
<lushnikov at chromium.org>'s request for review:
Bug 106705: Web Inspector: add text editor formatter test
https://bugs.webkit.org/show_bug.cgi?id=106705

Attachment 182582: Patch
https://bugs.webkit.org/attachment.cgi?id=182582&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=182582&action=review


> LayoutTests/inspector/editor/editor-test.js:22
> +function enter()

Please declare it inline

> LayoutTests/inspector/editor/text-editor-formatter.html:21
> +	   for(var i = 0; i < codeLines.length; ++i) {

Remove {}

> LayoutTests/inspector/editor/text-editor-formatter.html:54
> +    function bar() {

{ on the next line