Allow cancellation of in progress synchronization

RESOLVED FIXED in FxOS-S9 (16Oct)

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ferjm, Assigned: ferjm)

Tracking

unspecified
FxOS-S9 (16Oct)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
The Sync app should be able to cancel an on going synchronization and the SyncManager should be able to request this cancellation. Cases where a cancellation is needed:

- The user disables Sync while a synchronization is in progress.
- The user logs out from FxA while a synchronization is in progress.
- We add a specific button in the Settings UI to allow the user to cancel an on going synchronization.
(Assignee)

Updated

3 years ago
Blocks: 824026
Target Milestone: --- → FxOS-S8 (02Oct)
(Assignee)

Updated

3 years ago
Assignee: nobody → ferjmoreno
(Assignee)

Updated

3 years ago
Target Milestone: FxOS-S8 (02Oct) → FxOS-S9 (16Oct)
(Assignee)

Updated

3 years ago
Target Milestone: FxOS-S9 (16Oct) → FxOS-S8 (02Oct)
Created attachment 8669012 [details] [review]
[gaia] ferjm:bug1205239.cancelsync > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Target Milestone: FxOS-S8 (02Oct) → FxOS-S9 (16Oct)
(Assignee)

Comment 2

3 years ago
https://github.com/mozilla-b2g/gaia/commit/a3e13e4bea5fdbea870072af1a5b8f0cb03633b4
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

3 years ago
Ugh, wrong bug
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

3 years ago
Attachment #8669012 - Flags: review?(mbdejong)
Comment on attachment 8669012 [details] [review]
[gaia] ferjm:bug1205239.cancelsync > mozilla-b2g:master

Looks solid! :) I saw some "XXX FIXME : Dispatch a mozFxAccountsChromeEvent: undefined" being logged while trying it out, but it all seems to work.

I did notice that if I disable my WiFi and then click "Sync Now", it also disables sync (which should not happen), but that's probably unrelated to this PR.
Attachment #8669012 - Flags: review?(mbdejong) → review+
(Assignee)

Comment 5

3 years ago
(In reply to Michiel de Jong [:michielbdejong] from comment #4)
> Comment on attachment 8669012 [details] [review]
> [gaia] ferjm:bug1205239.cancelsync > mozilla-b2g:master
> 
> Looks solid! :) I saw some "XXX FIXME : Dispatch a mozFxAccountsChromeEvent:
> undefined" being logged while trying it out, but it all seems to work.
> 

That's a false warning with chrome events without a type property https://mxr.mozilla.org/mozilla-central/source/b2g/components/SystemAppProxy.jsm#128. That's an old check from when we used more generic mozChromEvents. It's totally harmless.

> I did notice that if I disable my WiFi and then click "Sync Now", it also
> disables sync (which should not happen), but that's probably unrelated to
> this PR.

Oh, that's a bug then.
(Assignee)

Comment 7

3 years ago
https://github.com/mozilla-b2g/gaia/commit/0f114c02efca8e5e1d0a3fcac7a2b8e8e6957701
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.