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

Issue 6726956523454464: Issue 153 - Reimplement io.js based on OS.File API (Closed)

Created:
March 21, 2014, 1:10 p.m. by Wladimir Palant
Modified:
March 21, 2014, 5:39 p.m.
Reviewers:
Felix Dahlke
Visibility:
Public.

Description

The new API is considerably simpler than the old approaches and moves all I/O off the main thread, not just read and write operations. The changes as far as our API is concerned: * Reading from URL is no longer possible (consistent with the Chrome implementation). * The encode/decode parameter is unused, UTF-8 encoding is always being used now (consistent with the Chrome implementation). * defaults/patterns.ini is no longer being used as fallback, instead FilterStorage.firstRun flag is being set (allows removing our current hack in Chrome and implement detection of missing patterns.ini properly).

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+108 lines, -149 lines) Patch
M .hgsubstate View 1 chunk +1 line, -1 line 0 comments Download
M lib/filterStorage.js View 2 chunks +11 lines, -2 lines 0 comments Download
M lib/io.js View 4 chunks +91 lines, -141 lines 1 comment Download
M metadata.gecko View 1 chunk +5 lines, -5 lines 0 comments Download

Messages

Total messages: 2
Wladimir Palant
March 21, 2014, 1:10 p.m. (2014-03-21 13:10:24 UTC) #1
Felix Dahlke
March 21, 2014, 3:44 p.m. (2014-03-21 15:44:26 UTC) #2

Powered by Google App Engine
This is Rietveld