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

Issue 29450608: Issue 5279 - Fix broken I/O in non-Firefox applications (Closed)

Created:
May 29, 2017, 11:54 a.m. by Wladimir Palant
Modified:
May 29, 2017, 12:56 p.m.
Reviewers:
Felix Dahlke
Base URL:
https://hg.adblockplus.org/adblockplus
Visibility:
Public.

Description

Issue 5279 - Fix broken I/O in non-Firefox applications

Patch Set 1 #

Total comments: 4

Patch Set 2 : Addressed comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -38 lines) Patch
M lib/io.js View 1 4 chunks +74 lines, -38 lines 0 comments Download

Messages

Total messages: 4
Wladimir Palant
May 29, 2017, 11:54 a.m. (2017-05-29 11:54:39 UTC) #1
Felix Dahlke
Looks a _lot_ better than before! https://codereview.adblockplus.org/29450608/diff/29450609/lib/io.js File lib/io.js (right): https://codereview.adblockplus.org/29450608/diff/29450609/lib/io.js#newcode249 lib/io.js:249: "readFromFile", "copyFile", "renameFile", ...
May 29, 2017, 12:22 p.m. (2017-05-29 12:22:30 UTC) #2
Wladimir Palant
https://codereview.adblockplus.org/29450608/diff/29450609/lib/io.js File lib/io.js (right): https://codereview.adblockplus.org/29450608/diff/29450609/lib/io.js#newcode249 lib/io.js:249: "readFromFile", "copyFile", "renameFile", "removeFile", "statFile" On 2017/05/29 12:22:30, Felix ...
May 29, 2017, 12:41 p.m. (2017-05-29 12:41:23 UTC) #3
Felix Dahlke
May 29, 2017, 12:51 p.m. (2017-05-29 12:51:09 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld