Closed Bug 916545 Opened 11 years ago Closed 11 years ago

Simplify the client IO (sync vs. async)

Categories

(L20n :: JS Library, defect)

Product:
L20n
Component:
JS Library
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: stas, Assigned: stas)

References

Details

Attachments

(1 file)

Use onreadystatechange
1.83 KB, patch
zbraniecki
: review+
Details | Diff | Splinter Review 
The load and loadSync could be combined into one method.
onreadystatechange has better support as it's part of XHR1, where's load is part of XHR2: http://www.w3.org/TR/XMLHttpRequest2/

This is also in line with the change proposed in bug 868852.
Assignee: nobody → stas
Status: NEW → ASSIGNED
Attachment #805028 - Flags: review?(gandalf)
Attachment #805028 - Flags: review?(gandalf) → review+
Blocks: 917195
https://github.com/l20n/l20n.js/commit/22b2469ef82e0584b2767771c5d1892826ed9398
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: