Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/ui/filters-filterview.js

Issue 4884233277407232: Issue 2257 - Replaced non-standard function expressions with ES6 arrow functions (Closed)
Patch Set: 2nd argument to Utils.runAsync isn't necessary with arrow functions Created April 1, 2015, 5:47 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/content/ui/filters-filterview.js
===================================================================
--- a/chrome/content/ui/filters-filterview.js
+++ b/chrome/content/ui/filters-filterview.js
@@ -502,7 +502,7 @@
let oldCount = this.rowCount;
if (this._subscription && this._subscription.filters.length)
{
- this.data = this._subscription.filters.map(function(f, i) ({index: i, filter: f}));
+ this.data = this._subscription.filters.map((f, i) => ({index: i, filter: f}));
if (this.sortProc)
{
// Hide comments in the list, they should be sorted like the filter following them
@@ -830,20 +830,20 @@
filter.disabled = !filter.disabled;
},
- isContainer: function(row) false,
- isContainerOpen: function(row) false,
- isContainerEmpty: function(row) true,
- getLevel: function(row) 0,
- getParentIndex: function(row) -1,
- hasNextSibling: function(row, afterRow) false,
- toggleOpenState: function(row) {},
- getProgressMode: function() null,
- getImageSrc: function() null,
- isSeparator: function() false,
- performAction: function() {},
- performActionOnRow: function() {},
- performActionOnCell: function() {},
- getCellValue: function() null,
- setCellValue: function() {},
- selectionChanged: function() {},
+ isContainer: row => false,
+ isContainerOpen: row => false,
+ isContainerEmpty: row => true,
+ getLevel: row => 0,
+ getParentIndex: row => -1,
+ hasNextSibling: (row, afterRow) => false,
+ toggleOpenState: row => {},
+ getProgressMode: () => null,
+ getImageSrc: () => null,
+ isSeparator: () => false,
+ performAction: () => {},
+ performActionOnRow: () => {},
+ performActionOnCell: () => {},
+ getCellValue: () => null,
+ setCellValue: () => {},
+ selectionChanged: () => {}
};

Powered by Google App Engine
This is Rietveld