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

Issue 29332044: Issue 3381 - notification unit tests are broken (Closed)

Created:
Dec. 7, 2015, 3:27 p.m. by Erik
Modified:
Dec. 28, 2015, 12:14 p.m.
Visibility:
Public.

Description

Issue 3381 - notification unit tests are broken simply converted global `let` scoped variables to use `var` instead of `let`.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -14 lines) Patch
M lib/notification.js View 2 chunks +14 lines, -14 lines 0 comments Download

Messages

Total messages: 3
Erik
Dec. 7, 2015, 3:33 p.m. (2015-12-07 15:33:32 UTC) #1
Wladimir Palant
LGTM
Dec. 9, 2015, 4:25 p.m. (2015-12-09 16:25:17 UTC) #2
Thomas Greiner
Dec. 11, 2015, 4:45 p.m. (2015-12-11 16:45:46 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld