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

Issue 6519778499887104: Issue 2010 - Remove some uses of "for each" with "for of" (Closed)

Created:
Feb. 18, 2015, 1:32 p.m. by tschuster
Modified:
Feb. 28, 2015, 12:44 p.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Issue 2010 - Remove some uses of "for each" with "for of"

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -24 lines) Patch
M chrome/content/errors.html View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/composer.js View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/filters.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/filters-filterview.js View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/content/ui/filters-subscriptionview.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/content/ui/sendReport.js View 1 3 chunks +11 lines, -11 lines 0 comments Download
M chrome/content/ui/sidebar.js View 1 5 chunks +7 lines, -7 lines 0 comments Download

Messages

Total messages: 3
tschuster
Feb. 26, 2015, 11:50 p.m. (2015-02-26 23:50:35 UTC) #1
Wladimir Palant
LGTM
Feb. 27, 2015, 7:50 p.m. (2015-02-27 19:50:16 UTC) #2
tschuster
Feb. 28, 2015, 12:44 p.m. (2015-02-28 12:44:30 UTC) #3
On 2015/02/27 19:50:16, Wladimir Palant wrote:
> LGTM

Thanks Wladimir.

Powered by Google App Engine
This is Rietveld