1111057 - Make MobileConnectionCallback consistently async

Status: RESOLVED DUPLICATE of bug 1070831

(Firefox OS Graveyard :: RIL, defect)

Description

[pgravel]

Most MobileConnectionCallback function have a built-in asynchronous step, except for the error cases that result in FireDetailedError, which makes behavior a little inconsistent when issuing callbacks.
http://mxr.mozilla.org/mozilla-central/source/dom/mobileconnection/MobileConnectionCallback.cpp#387

Requesting that an async step be added for the error callbacks that use FireDetailedError()

Comment 1

[Hsin-Yi Tsai (she/her) [:hsinyi]]

(In reply to pgravel from comment #0)
> Most MobileConnectionCallback function have a built-in asynchronous step,
> except for the error cases that result in FireDetailedError, which makes
> behavior a little inconsistent when issuing callbacks.
> http://mxr.mozilla.org/mozilla-central/source/dom/mobileconnection/
> MobileConnectionCallback.cpp#387
> 
> Requesting that an async step be added for the error callbacks that use
> FireDetailedError()

Hi,
The section of code is going to be removed in https://bugzilla.mozilla.org/attachment.cgi?id=8536486
After that, I think your concern will be released.

Comment 2

[Hsin-Yi Tsai (she/her) [:hsinyi]]

I am going to close this bug as duplicate of bug 1070831. Feel free to reopen it if i misunderstand something or you disagree, thank you.