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

Issue 10214006: Added JS-based XMLHttpRequest implementation (Closed)

Created:
April 15, 2013, 11:36 a.m. by Wladimir Palant
Modified:
May 7, 2013, 7:36 a.m.
Reviewers:
Felix Dahlke, Oleksandr
Visibility:
Public.

Description

Added JS-based XMLHttpRequest implementation

Patch Set 1 #

Patch Set 2 : Unbitrotted patch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+175 lines, -10 lines) Patch
M lib/compat.js View 1 2 chunks +124 lines, -10 lines 0 comments Download
M test/WebRequest.cpp View 1 1 chunk +51 lines, -0 lines 0 comments Download

Messages

Total messages: 3
Wladimir Palant
April 15, 2013, 11:36 a.m. (2013-04-15 11:36:22 UTC) #1
Felix Dahlke
LGTM
April 16, 2013, 2:33 p.m. (2013-04-16 14:33:51 UTC) #2
Oleksandr
April 16, 2013, 2:43 p.m. (2013-04-16 14:43:17 UTC) #3
On 2013/04/16 14:33:51, Felix H. Dahlke wrote:
> LGTM
LGTM, though I don't quite see what is this for :)

Powered by Google App Engine
This is Rietveld