Closed Bug 965020 Opened 6 years ago Closed 6 years ago

Use task affinities more clearly to avoid some back-stack issues

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 29

People

(Reporter: rnewman, Assigned: rnewman)

References

(Blocks 1 open bug)

Details

(Whiteboard: [qa+])

Attachments

(1 file)

The lesser version of Bug 964883.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Comment on attachment 8366956 [details] [diff] [review]
Use task affinities more clearly to avoid some back-stack issues.

wesj reviewed this over my shoulder.
Attachment #8366956 - Flags: review+
https://hg.mozilla.org/services/services-central/rev/d757713c39cc
Whiteboard: [qa+] → [qa+][fixed in services]
I wanted to test this to see if it affects webapps in some way, so I applied the patch to my trunk clone and got this build error:

 2:39.56 /Users/myk/Mozilla/gecko-dev/mobile/android/base/preferences/SyncPreference.java:25: invalid method declaration; return type required
 2:39.56     private openSync11Settings() {

Perhaps it depends on something in the Services branch?
(In reply to Myk Melez [:myk] [@mykmelez] from comment #4)
> I wanted to test this to see if it affects webapps in some way, so I applied
> the patch to my trunk clone and got this build error:


That's just a typo in the patch attached to the bug (missing qref). Do this instead:

  hg qimport -n taskaffinity -P https://hg.mozilla.org/services/services-central/rev/d757713c39cc
https://hg.mozilla.org/mozilla-central/rev/d757713c39cc
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [qa+][fixed in services] → [qa+]
Target Milestone: --- → Firefox 29
You need to log in before you can comment on or make changes to this bug.