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

Issue 5743717579227136: Get rid of Utils.threadManager (Closed)

Created:
March 13, 2014, 11:49 a.m. by Wladimir Palant
Modified:
March 13, 2014, 4:58 p.m.
Reviewers:
Thomas Greiner
Visibility:
Public.

Description

This replaces our Utils.threadManager property by the built-in Services.tm property. This also replaces several unnecessary uses of the thread manager by Utils.runAsync().

Patch Set 1 #

Patch Set 2 : Removed a line that is no longer in use #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -37 lines) Patch
M chrome/content/ui/sendReport.js View 2 chunks +2 lines, -2 lines 0 comments Download
M lib/requestNotifier.js View 1 1 chunk +22 lines, -33 lines 0 comments Download
M lib/utils.js View 2 chunks +1 line, -2 lines 0 comments Download

Messages

Total messages: 3
Wladimir Palant
March 13, 2014, 11:49 a.m. (2014-03-13 11:49:32 UTC) #1
Wladimir Palant
Removed a line that is no longer in use
March 13, 2014, 11:52 a.m. (2014-03-13 11:52:29 UTC) #2
Thomas Greiner
March 13, 2014, 4:43 p.m. (2014-03-13 16:43:31 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld