Closed
Bug 625917
Opened 13 years ago
Closed 13 years ago
e10sify url module
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b2
People
(Reporter: myk, Assigned: myk)
Details
Attachments
(1 file)
The url module should be e10sified, so it works in e10s mode. Here's an e10s adapter for it. Note: this is a branch off of the propagate-exceptions branch, as the adapter relies on exception propagation to work correctly.
Assignee | ||
Comment 1•13 years ago
|
||
Associated pull request: https://github.com/mozilla/addon-sdk/pull/89
Assignee: nobody → myk
Status: NEW → ASSIGNED
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Target Milestone: --- → 1.0b2
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #503995 -
Flags: review?(warner-bugzilla)
Assignee | ||
Updated•13 years ago
|
Attachment #503995 -
Attachment is patch: false
Attachment #503995 -
Attachment mime type: text/plain → text/html
Comment 3•13 years ago
|
||
Comment on attachment 503995 [details]
pull request 89
approach looks sound, basic manual tests pass, 'cfx test --e10s' passes. Good by me.
Attachment #503995 -
Flags: review?(warner-bugzilla) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Topic commit: https://github.com/mozilla/addon-sdk/commit/0acff5437ca7246c53346f90d87d508a97bd17d6 Commit to address review comments (from IRC): https://github.com/mozilla/addon-sdk/commit/a47ef70ca8f87c995f8969a17f386818b8050ff7 Merge commit: https://github.com/mozilla/addon-sdk/commit/592b330dac1e7258d24928de696bb24d96a738d7
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•