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

Issue 10233013: Crawler, second version (Closed)

Created:
April 12, 2013, 1:38 p.m. by Eric
Modified:
Dec. 30, 2014, 1:46 p.m.
Visibility:
Public.

Description

Crawler, second version. This is everything from my start with this project. Base revision is 74. This constitutes revision range 75:336.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8874 lines, -286 lines) Patch
M .hgignore View 1 chunk +2 lines, -0 lines 0 comments Download
M .hgsubstate View 1 chunk +1 line, -1 line 0 comments Download
M README.md View 1 chunk +2 lines, -3 lines 0 comments Download
R chrome/content/crawler.js View 1 chunk +0 lines, -74 lines 0 comments Download
M chrome/content/crawler.xul View 1 chunk +202 lines, -31 lines 0 comments Download
A chrome/content/crawler_ui.js View 1 chunk +363 lines, -0 lines 0 comments Download
A chrome/content/sandbox_ui.js View 1 chunk +71 lines, -0 lines 0 comments Download
A chrome/content/task_test_ui.js View 1 chunk +109 lines, -0 lines 0 comments Download
M chrome/locale/en-US/crawler.dtd View 1 chunk +2 lines, -2 lines 0 comments Download
A chrome/skin/dialog-cancel-3.png View Binary file 0 comments Download
A chrome/skin/folder.png View Binary file 0 comments Download
A chrome/skin/text-x-source.png View Binary file 0 comments Download
A lib/action.js View 1 chunk +882 lines, -0 lines 0 comments Download
A lib/action_platform.js View 1 chunk +77 lines, -0 lines 0 comments Download
A lib/application.js View 1 chunk +234 lines, -0 lines 0 comments Download
A lib/bootstrap_xpcom.js View 1 chunk +117 lines, -0 lines 0 comments Download
A lib/browser.js View 1 chunk +468 lines, -0 lines 0 comments Download
M lib/client.js View 1 chunk +2 lines, -2 lines 0 comments Download
A lib/command_line.js View 1 chunk +134 lines, -0 lines 0 comments Download
A lib/counter_task.js View 1 chunk +117 lines, -0 lines 0 comments Download
M lib/crawler.js View 1 chunk +635 lines, -103 lines 0 comments Download
A lib/encoding.js View 1 chunk +1145 lines, -0 lines 0 comments Download
A lib/instruction.js View 1 chunk +472 lines, -0 lines 0 comments Download
A lib/logger.js View 1 chunk +142 lines, -0 lines 0 comments Download
M lib/main.js View 1 chunk +224 lines, -36 lines 0 comments Download
M lib/storage.js View 1 chunk +200 lines, -32 lines 0 comments Download
A lib/task.js View 1 chunk +393 lines, -0 lines 0 comments Download
A lib/yaml.js View 1 chunk +2068 lines, -0 lines 0 comments Download
M metadata.gecko View 1 chunk +1 line, -2 lines 0 comments Download
A test/action_platform-test.js View 1 chunk +42 lines, -0 lines 0 comments Download
A test/jsTestDriver.conf View 1 chunk +6 lines, -0 lines 0 comments Download
A test/test-action.js View 1 chunk +763 lines, -0 lines 0 comments Download

Messages

Total messages: 1
Eric
April 19, 2013, 12:58 p.m. (2013-04-19 12:58:48 UTC) #1 
OK. Now I know. This review has been up for a week.

This is basically all the code. There are places where the code is known to be
rough, since they weren't critical or were changing rapidly.

Powered by Google App Engine
This is Rietveld