[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 #14 from Rahul Tiwari <rahultiwari.cse.iitr at gmail.com>  2012-06-26 04:34:58 PST ---
(In reply to comment #13)
> (From update of attachment 149447 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=149447&action=review
> 
> > Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:278
> > +        for (var i = 0; i < this.watchExpressions.length; ++i)
> 
> style nit: {} around multiline body

Thanks Yury for the review. I was intending to use the same styling as you proposed, but noticed that the function just above it "findAddedTreeElement" also omitted the braces. So to keep the code consistent, I omitted the braces as well.
So should I change it?

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