[Webkit-unassigned] [Bug 89735] Web Inspector: Provide context menu 'Delete all watch expressions.'

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=89735





--- Comment #10 from Andrey Adaikin <aandrey at chromium.org>  2012-06-25 10:25:48 PST ---
(From update of attachment 149261)
View in context: https://bugs.webkit.org/attachment.cgi?id=149261&action=review

> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:263
> +        this.watchExpressions.splice(0, expressionlength);

this.watchExpressions = []; ?

> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:278
> +        var count= 0;

spaces around "="

> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:279
> +        for (var i = 0; i < this.watchExpressions.length; ++i){

spaces between "){"

> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:379
> +       var expressioncount= this.treeOutline.section._expressionCount();

spaces; expressioncount => expressionCount

> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:381
> +            contextMenu.appendItem(WebInspector.UIString("Delete all watch expressions"), this._deleteAllExpressions.bind(this));

wrong indent

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.