Closed Bug 1026005 Opened 8 years ago Closed 8 years ago

Create AsyncTaskLoader for querying/updating Fennec Accounts status


(Firefox for Android Graveyard :: General, defect)

Not set


(Not tracked)

Firefox 34


(Reporter: nalexander, Assigned: nalexander)




(1 file, 1 obsolete file)

This is the Fragment-centric way to establish and update UI in response to accounts coming, going, and changing status.
This is needed in the Fennec UI (for the Remote Tabs panel), but also in the background services UI (for basically all activities, eventually).  Therefore we'll implement in android-sync, but expose in the FirefoxAccounts module.
rnewman: your queue is long.  Maybe delegate to mcomella?  Or some other front-end person who would know about loaders?
Attachment #8440816 - Flags: review?(rnewman)
Comment on attachment 8440816 [details] [review]
Link to Github pull-request:

It's review day for mcomella.
Attachment #8440816 - Flags: review?(rnewman) → review?(michael.l.comella)
Comment on attachment 8440816 [details] [review]
Link to Github pull-request:

Comments on github.
Attachment #8440816 - Flags: review?(michael.l.comella) → review-
nalexander (and anyone keeping score): checking status ...

In our etherpad discussion (, you mention a loader wip for this bug ... I don't use / am unfamiliar with git as a matter of course, but I'm assuming you're referring to these two changesets, and that the others were handled in their respective bugzillas:

Also, in the etherpad you mentioned wanting to do (a third?) bit:

   "The states that the account can be in are the members of Action:  I'd like to move that enum into (it's part of the contract between background services and Fennec)"

Would that be part of this patch? Should I open a new one?

I'm trying to get a handle around each puzzle piece. Some of the planning/direction under Meta Bug 1002573 - "Improve Fennec's Remote Tabs panel" seems to be cross bleeding, and I like to avoid scope-creep  :-)
This is about as long as I've taken to get to a review, I think :)  Thanks for catching that thread handling issue!
Attachment #8474906 - Flags: review?(michael.l.comella)
Comment on attachment 8474906 [details] [review]
Link to Github pull-request:

r=mcomella via github and IRC.
Attachment #8474906 - Flags: review?(michael.l.comella) → review+
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.