1215604 - Fennec firstrun SyncPanel does not launch web activity correctly

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Nick Alexander :nalexander [he/him]]

There are two parts to this ticket:

First, https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/firstrun/SyncPanel.java#32 launches the native Get Started activity directly.  This is probably a rebasing issue, since the world has moved on and we need to delegate (using the mechanism of Bug 1204937).  An example is at https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/firstrun/WelcomePanel.java#31.

Second, there's another mis-leading usage of the Get Started native activity, although it would only be shown to Sync 1.1 users: https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/sync/setup/activities/SendTabActivity.java#258.

The over-arching issue is that we need to /remove/ the manifest entries for native activities when they aren't used, which would crash offending users like this one when the web flow is used.  This was intentionally a staged process; we'll pull the trigger in Bug 1192004.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1215604 - Use <intent-filter> indirection to allow native and web Sync Get Started activities. r?liuche

Bug 1215604 - Use <intent-filter> indirection to allow native and web Sync Get Started activities. r?liuche

Comment 2

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8677143 [details]
MozReview Request: Bug 1215604 - Use <intent-filter> indirection to allow native and web Sync Get Started activities. r?liuche

margaret: no reason to wait for liuche for this.  Steal if you can, please.

Comment 3

[:Margaret Leibovic]

Comment on attachment 8677143 [details]
MozReview Request: Bug 1215604 - Use <intent-filter> indirection to allow native and web Sync Get Started activities. r?liuche

https://reviewboard.mozilla.org/r/22895/#review20625

Comment 4

[Nick Alexander :nalexander [he/him]]

https://hg.mozilla.org/integration/fx-team/rev/ebe762936d7aa9d7b3bdb586d6ff99b4196488d8
Bug 1215604 - Use <intent-filter> indirection to allow native and web Sync Get Started activities. r=margaret

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ebe762936d7a