Simplify the client IO (sync vs. async)

RESOLVED FIXED

Status

L20n
JS Library
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: stas, Assigned: stas)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The load and loadSync could be combined into one method.
(Assignee)

Comment 1

4 years ago
Created attachment 805028 [details] [diff] [review]
Use onreadystatechange

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+
(Assignee)

Updated

4 years ago
Blocks: 917195
(Assignee)

Comment 2

4 years ago
https://github.com/l20n/l20n.js/commit/22b2469ef82e0584b2767771c5d1892826ed9398
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.