829770 - Ensure Services.common spinning callback technique plays well with WinRT backend

Status: RESOLVED WORKSFORME

(Firefox for Metro Graveyard :: Sync, defect)

Description

[Jim Mathies [:jimm]]

Spun off from bug 809171. See bug 774779. We need to be sure the winrt backend is working ok with the internal loop.

http://mxr.mozilla.org/mozilla-central/source/services/common/async.js#89

Comment 1

[Richard Newman [:rnewman]]

Note that we won't have addressed Sync's event loop spinning until probably the end of Q1, maybe later, and it's possible we won't be able to eliminate all uses of it.

Also note that some other code -- e.g., the current version of Firefox Health Report -- spins the event loop using both this technique and this precise code.

If this bug involves work that isn't Metro-specific -- that is, if it touches /services/common -- please move it to Mozilla Services :: Common and explain the issue.

Finally, did you mean to block Bug 600059, or depend on it?

Comment 2

[Jim Mathies [:jimm]]

(In reply to Richard Newman [:rnewman] from comment #1)
> Note that we won't have addressed Sync's event loop spinning until probably
> the end of Q1, maybe later, and it's possible we won't be able to eliminate
> all uses of it.
> 
> Also note that some other code -- e.g., the current version of Firefox
> Health Report -- spins the event loop using both this technique and this
> precise code.
> 
> If this bug involves work that isn't Metro-specific -- that is, if it
> touches /services/common -- please move it to Mozilla Services :: Common and
> explain the issue.

Thanks for the info.

> Finally, did you mean to block Bug 600059, or depend on it?

Neither, I punched in the wrong bug number.

Comment 3

[Jim Mathies [:jimm]]

I've been testing message processing quite a bit and so far haven't seen any issues.