If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[System2] System service proxy should deal with promise in promise

RESOLVED FIXED

Status

Firefox OS
Gaia::System
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: alive, Assigned: alive)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

System is proxying the service with a promise already;
if the proxy-ed service is returning another promise,
the caller need to call then again to get the real promise.
Created attachment 8515904 [details]
Patch (including bug 1062819)

Promise is profound.... :)
Attachment #8515904 - Flags: review?(etienne)
Comment on attachment 8515904 [details]
Patch (including bug 1062819)

r=me with the comment addressed, happy with the change itself :)
Attachment #8515904 - Flags: review?(etienne) → review+
Assignee: nobody → alive
A rebased version
https://github.com/mozilla-b2g/gaia/pull/25917

Waiting for tbpl run
https://github.com/mozilla-b2g/gaia/commit/779f05fead3d009f6e7fe713ad0fea16b6f2fb31
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.