[meta] Android Sync 0.4 code drop

VERIFIED FIXED in Firefox 11

Status

Android Background Services
Android Sync
P1
normal
VERIFIED FIXED
6 years ago
4 years ago

People

(Reporter: rnewman, Assigned: rnewman)

Tracking

unspecified
mozilla12
ARM
Android
Dependency tree / graph

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Tracking what's going in to the next one.
(Assignee)

Updated

6 years ago
Depends on: 721305
(Assignee)

Updated

6 years ago
Depends on: 719693
(Assignee)

Updated

6 years ago
Depends on: 722485
(Assignee)

Comment 1

6 years ago
So far:

3ad57f4 Bug 718703 - Pair a Device crashes. r=rnewman
3c530d1 Bug 709660 - Fixed blurry icons used in SyncAdapter. r=rnewman
bde287a Bug 719693 - Compatibility for JDK < 6.
91c5e23 Bug 719693 - Effect review comments on 6ffb7d72f.
6ffb7d7 Bug 719693 - Handle date-format HTTP Retry-After headers.
6df289e Bug 709348 - Trace logging and fixed logging of visit equality.
71580df Bug 709348 - Include Fennec visits and counts in HistoryRecord androidID equivalence.
8f6a106 Bug 709348 - Equality work.
f40bf4a Bug 709348 - Part 5: Rework handling of certain cases of incoming deletion, including history visits.
0dd004d Bug 709348 - Part 4: Reworking equality tests between records.
2210717 Bug 709348 - Part 3: Better commenting for ignored bookmark records.
c05236b Bug 709348 - Part 2: Introduce trace log method.
a34f4aa Bug 709348 - Part 1: Introduce copyWithIDs to clone records.
796b15f Bug 721305 - sync.link.advancedsetup.label should use …, not ....
adf7586 Bump AndroidSync User-Agent version.

I will land at the same time (pending review, of course):

 Bug 718703 - Part 0: Makefile fix for preprocessing multiple files. r=ted
 Bug 722896.
 Bug 718703 - Part 1: Sync crashes from Pair a Device link in sync settings. r=rnewman
 Bug 718703 - Part 2: Makefile change to preprocess sync_options.xml. r=ted

I have a try build of these things:

  https://tbpl.mozilla.org/?tree=Try&rev=0e2d59ecbaf8

I will bump the rest of Bug 709348, and anything else left over, to the next drop.
(Assignee)

Updated

6 years ago
Depends on: 709348
(Assignee)

Updated

6 years ago
Depends on: 722434
(Assignee)

Updated

6 years ago
Depends on: 722426
(Assignee)

Updated

6 years ago
Depends on: 722430
(Assignee)

Comment 2

6 years ago
  Bug 722434 - check for failure after JSON parsing. r=rnewman
  Bug 722426: Remove WRITE_SECURE_SETTINGS from the Android manifest, since this permission is only granted to system/firmware applications. r=rnewman
  Bug 722896. r=dchan
  Bug 722430: Propagate NoSuchAlgorithmException and InvalidKeyException; wrap in CryptoException higher up the call stack. r=rnewman
Depends on: 722896, 718703
Target Milestone: --- → mozilla12

Comment 3

6 years ago
Try run for 9569e93be612 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=9569e93be612
Results (out of 131 total builds):
    exception: 30
    success: 67
    warnings: 16
    failure: 1
    other: 17
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/rnewman@mozilla.com-9569e93be612
(Assignee)

Updated

6 years ago
Priority: -- → P1
(Assignee)

Updated

6 years ago
Depends on: 722945
(Assignee)

Comment 4

6 years ago
Bug 709348 is in, as is Bug 722945.

Comment 5

6 years ago
Try run for 943421074305 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=943421074305
Results (out of 271 total builds):
    exception: 2
    success: 239
    warnings: 29
    failure: 1
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/rnewman@mozilla.com-943421074305
(Assignee)

Comment 6

6 years ago
https://hg.mozilla.org/mozilla-central/rev/34c03a42a465
https://hg.mozilla.org/mozilla-central/rev/b9b8107bb91f
https://hg.mozilla.org/mozilla-central/rev/fad9c50ab47d
https://hg.mozilla.org/mozilla-central/rev/50dfa77287fc
https://hg.mozilla.org/mozilla-central/rev/4b1a491ad571
https://hg.mozilla.org/mozilla-central/rev/2b49a406570b
https://hg.mozilla.org/mozilla-central/rev/e3427fb08d16
https://hg.mozilla.org/mozilla-central/rev/5a918bbf0cd6
https://hg.mozilla.org/mozilla-central/rev/dab232f3a4f3
https://hg.mozilla.org/mozilla-central/rev/8d922468a7fe
https://hg.mozilla.org/mozilla-central/rev/2ce9638b93e2
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Depends on: 718544, 723992, 723235, 720304, 723240
Resolution: --- → FIXED
(Assignee)

Comment 7

6 years ago
Created attachment 594284 [details] [diff] [review]
Approval coathook.

Hanging m-a and m-b approval on this. Some of the bugs that landed at the same time (particularly the ones that touched Makefiles, and thus didn't flow with everything else) have independent flags.

akeybl, please let me know if you have any questions.
Attachment #594284 - Flags: approval-mozilla-beta?
Attachment #594284 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 8

6 years ago
I have updated blassey's Aurora patch queue:

http://hg.mozilla.org/users/blassey_mozilla.com/patches-needed-on-aurora

The Beta patch queue is currently busted, so I haven't updated that yet.
(Assignee)

Comment 9

6 years ago
> The Beta patch queue is currently busted, so I haven't updated that yet.

I commented out the patches that were labeled in the series as failing to apply, and imported all of mine. Good to go.
This is on my radar, but we should probably wait to have it on m-c for a day before pushing to Aurora/Beta.
(Assignee)

Comment 11

6 years ago
(In reply to Alex Keybl [:akeybl] from comment #10)
> This is on my radar, but we should probably wait to have it on m-c for a day
> before pushing to Aurora/Beta.

Yup, and tchung and crew are testing this afternoon.
Comment on attachment 594284 [details] [diff] [review]
Approval coathook.

[Triage Comment]
Approved for Aurora 12 and Beta 11.
Attachment #594284 - Flags: approval-mozilla-beta?
Attachment #594284 - Flags: approval-mozilla-beta+
Attachment #594284 - Flags: approval-mozilla-aurora?
Attachment #594284 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 13

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/e6022b12e254
https://hg.mozilla.org/releases/mozilla-aurora/rev/6aa8343c3b4a
https://hg.mozilla.org/releases/mozilla-aurora/rev/99eca3dd8a7c
https://hg.mozilla.org/releases/mozilla-aurora/rev/d6581d4115cf
https://hg.mozilla.org/releases/mozilla-aurora/rev/43ee85dffc7b
https://hg.mozilla.org/releases/mozilla-aurora/rev/769552280aa4
https://hg.mozilla.org/releases/mozilla-aurora/rev/600a1f879378
https://hg.mozilla.org/releases/mozilla-aurora/rev/b07aec0d378b
https://hg.mozilla.org/releases/mozilla-aurora/rev/4f769f1fcf82
https://hg.mozilla.org/releases/mozilla-aurora/rev/badab638e09d
https://hg.mozilla.org/releases/mozilla-aurora/rev/13d4e59381f6
https://hg.mozilla.org/releases/mozilla-aurora/rev/64d39b99f7a4
status-firefox12: --- → fixed
calling this verified per depends bugs.

But what about the crypto meta bug 722485
Status: RESOLVED → VERIFIED
(Assignee)

Comment 15

6 years ago
(In reply to Tracy Walker [:tracy] from comment #14)
> calling this verified per depends bugs.
> 
> But what about the crypto meta bug 722485

Nearly done… and mostly [qa-], too.
Looks to already have landed on beta as well:

https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?changeset=088774d5dd19
status-firefox11: --- → fixed
Component: Android Sync → Android Sync
Product: Mozilla Services → Android Background Services
You need to log in before you can comment on or make changes to this bug.