Closed
Bug 551362
Opened 14 years ago
Closed 13 years ago
Review the XHR module, jetpack-core/lib/xhr.js
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: adw, Unassigned)
References
Details
http://hg.mozilla.org/labs/jetpack-sdk/file/tip/packages/jetpack-core/lib/xhr.js Module description from the docs: The xhr module provides access to XMLHttpRequest functionality, also known as AJAX.
Comment 1•14 years ago
|
||
As a first comment, the XHR module requires the use of |let v = new CTor| vs the favored |let v = CTor|
Comment 2•14 years ago
|
||
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product. To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
Comment 3•13 years ago
|
||
Given the length of time this code has been in the tree, and the exposure it has received, it doesn't seem like additional review at this point is worth the cost, with the exception of the cuddlefish module, about which concerns have been raised, so closing these bugs WONTFIX, except for that one.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•