Unified Diff: chrome/content/ui/filters-subscriptionview.js
Issue 6519778499887104:
Issue 2010 - Remove some uses of "for each" with "for of" (Closed)
Patch Set:
Created Feb. 26, 2015, 11:50 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« chrome/content/ui/filters-filterview.js ('k')
|
chrome/content/ui/sendReport.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: chrome/content/ui/filters-subscriptionview.js |
| =================================================================== |
| --- a/chrome/content/ui/filters-subscriptionview.js |
| +++ b/chrome/content/ui/filters-subscriptionview.js |
| @@ -82,17 +82,17 @@ ListManager.prototype = |
| // Remove existing entries if any |
| while (this._list.firstChild) |
| this._list.removeChild(this._list.firstChild); |
| // Now add all subscriptions |
| let subscriptions = FilterStorage.subscriptions.filter(this._filter, this); |
| if (subscriptions.length) |
| { |
| - for each (let subscription in subscriptions) |
| + for (let subscription of subscriptions) |
| this.addSubscription(subscription, null); |
| // Make sure first list item is selected after list initialization |
| Utils.runAsync(function() |
| { |
| this._list.selectItem(this._list.getItemAtIndex(this._list.getIndexOfFirstVisibleRow())); |
| }, this); |
| } |
« no previous file with comments
|
« chrome/content/ui/filters-filterview.js ('k')
|
chrome/content/ui/sendReport.js » ('j')
|
no next file with comments »
