780279 - Request a sync after persisting local tabs

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: Data Providers, defect, P2)

Description

[:Margaret Leibovic]

See bug 730039 comment 24. We just want to split off this functionality from the patch in that bug.

Comment 1

[Richard Newman [:rnewman]]

Very soon you'll be able to do something like:

  Account[] accounts = null;
  try {
    accounts = SyncAccounts.syncAccounts(context);
  } catch (org.mozilla.gecko.sync.CredentialException e) {
    // ...
  }

in addition to the facilities that SyncAccounts already provides.

(See <https://github.com/mozilla-services/android-sync/pull/239>.)

Comment 2

[Nick Alexander :nalexander [he/him]]

Attachment: patch against m-i

Needs Sync convenience method from https://github.com/mozilla-services/android-sync/pull/244 as well.

Comment 3

[Nick Alexander :nalexander [he/him]]

STR:

1. pair Android device to existing Sync account.
2. verify Sync is working.
3. in Fennec, add several tabs (3 or so works for me) and then delete some of them.
4. verify that a sync was requested by checking the logs for:

...
I FxSync(9158)                fennec_ncalexan :: SyncAdapter :: Forced sync: overruling remaining backoff of 83096128ms.
...
I FxSync(9158)                fennec_ncalexan :: Utils :: Asked to sync 'clients, tabs' and to skip ''.
...

NOTE: until Bug 728612 lands, you won't actually see outgoing tabs.  I implemented this ticket first to make it easier to test 728612.

Comment 4

[Nick Alexander :nalexander [he/him]]

This is no longer relevant.