1329152 - Custom tabs: Enable custom tabs by default (when released)

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P1)

Description

[Joe Cheng [:jcheng] (please needinfo)]

As Mozilla, we want users who set Firefox for Android as the default browser to also use Firefox when 3rd party apps use custom tabs

Comment 1

[Sebastian Kaspari (:sebastian; :pocmo)]

This is not controlled by us. However this should be the default behavior when the custom tabs support library (for third-party apps) is used.

Comment 2

[Julian Chu [:walkingice]]

Attachment: Bug 1329152 - Enable custom tabs by default

Remove related options, just use CustomTabs directly.

Review commit: https://reviewboard.mozilla.org/r/128384/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/128384/

Comment 3

[Julian Chu [:walkingice]]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/128384/diff/1-2/

Comment 4

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

https://reviewboard.mozilla.org/r/128384/#review131966

Technically this patch is okay. But there are two things that prevent me from r+ing:

* There are still open bugs linked to the MVP bug. Is this really ready? Has this been tested thoroughly by QA and do they recommend releasing?
* Please think about moving this feature behind a switchboard flag. This will allow a staged rollout and we will be able to pull this feature if we see problems. In fact that's something we should consider for all new features - especially with Aurora going away. In this case we might not be able to do this for the manifest entries - but we could keep the routing in LauncherActivity: If enabled (via switchboard) open custom tab, otherwise open browser.

Comment 5

[Julian Chu [:walkingice]]

(In reply to Sebastian Kaspari (:sebastian) from comment #4)
> Comment on attachment 8856432 [details]
> Bug 1329152 - Enable custom tabs by default
> 
> https://reviewboard.mozilla.org/r/128384/#review131966
> 
> Technically this patch is okay. But there are two things that prevent me
> from r+ing:
> 
> * There are still open bugs linked to the MVP bug. Is this really ready? Has
> this been tested thoroughly by QA and do they recommend releasing?

Per today's meeting we want to enable this then get more feedback(...bugs) as early as possible. Do you think this make sense?

> * Please think about moving this feature behind a switchboard flag. This
> will allow a staged rollout and we will be able to pull this feature if we
> see problems. In fact that's something we should consider for all new
> features - especially with Aurora going away. In this case we might not be
> able to do this for the manifest entries - but we could keep the routing in
> LauncherActivity: If enabled (via switchboard) open custom tab, otherwise
> open browser.

OK! I will prepare a new patch which involve Switchboard then we have ability to disable this. (just prepare the patch for technically reviewing, I won't land it until I get green light)

Comment 6

[Julian Chu [:walkingice]]

Attachment: Bug 1329152 - turn on CustomTabs via SwitchBoard

We want to enable CustomTabs by default. If we don't specify any value
in SwitchBoard, then CustomTabs function will be turned on.

Review commit: https://reviewboard.mozilla.org/r/133892/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/133892/

Comment 7

[Julian Chu [:walkingice]]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/128384/diff/2-3/

Comment 8

[Julian Chu [:walkingice]]

For now the patches are for technical review.

Comment 9

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

https://reviewboard.mozilla.org/r/128384/#review138412

Comment 10

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8861872 [details]
Bug 1329152 - turn on CustomTabs via SwitchBoard

https://reviewboard.mozilla.org/r/133892/#review138414

Please reverse this: The experiment should control whether a feature is enabled.

I added a new experiment called "custom-tabs" to the production configuration. This experiment is enabled for Nightly/Aurora and local builds already:
https://firefox.settings.services.mozilla.com/v1/buckets/fennec/collections/experiments/records

Comment 11

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

This is the last piece of P1 for custom tab.

Comment 12

[Julian Chu [:walkingice]]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/128384/diff/3-4/

Comment 13

[Julian Chu [:walkingice]]

Comment on attachment 8861872 [details]
Bug 1329152 - turn on CustomTabs via SwitchBoard

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/133892/diff/1-2/

Comment 14

[Julian Chu [:walkingice]]

(In reply to Sebastian Kaspari (:sebastian) from comment #10)
> Comment on attachment 8861872 [details]
> Bug 1329152 - be able to turn off CustomTabs via SwitchBoard
> 
> https://reviewboard.mozilla.org/r/133892/#review138414
> 
> Please reverse this: The experiment should control whether a feature is
> enabled.

OK! I reversed the statement. Now it enable CustomTabs via SwitchBoard.

Comment 15

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8861872 [details]
Bug 1329152 - turn on CustomTabs via SwitchBoard

https://reviewboard.mozilla.org/r/133892/#review148226

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3e06253547af
Enable custom tabs by default r=sebastian
https://hg.mozilla.org/integration/autoland/rev/3c03b94342b9
turn on CustomTabs via SwitchBoard r=sebastian

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/3e06253547af
https://hg.mozilla.org/mozilla-central/rev/3c03b94342b9

Comment 18

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Since we're not going to ship the GeckoApp impl. to beta, let's back this out.

Comment 19

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Bug 1371561 is for backing out this back.
We'll reopen current one when that lands.

Comment 20

[Carsten Book [:Tomcat]]

https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=27cad9749cddf68e11fdd4e5d73dad84a8f8cf23 is the backout push.

Comment 21

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/3c60b0f8edb0
Backed out changeset 3c03b94342b9

Comment 22

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

This is an assigned P1 bug without activity in two weeks. 

If you intend to continue working on this bug for the current release/iteration/sprint, remove the 'stale-bug' keyword.

Otherwise we'll reset the priority of the bug back to '--' on Monday, August 28th.

Comment 23

[Sebastian Kaspari (:sebastian; :pocmo)]

I wonder why this was backed out? All we would need to do is to enable/disable the switchboard experiment. No need to remove all the code.

Comment 24

[James Willcox (:snorp) ([e-mail]) (he/him)]

Wesly, Nevin, we need someone to do this, preferably before the merge tomorrow.

Comment 25

[James Willcox (:snorp) ([e-mail]) (he/him)]

I guess we can just land the attached patches and control via switchboard?

Comment 26

[Nevin Chen(Not active on Bugzilla)]

Hi Julian.
Please help with this.
Thank you!

Comment 27

[Julian Chu [:walkingice]]

Comment on attachment 8856432 [details]
Bug 1329152 - Enable custom tabs by default

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/128384/diff/4-5/

Comment 28

[Julian Chu [:walkingice]]

Comment on attachment 8861872 [details]
Bug 1329152 - turn on CustomTabs via SwitchBoard

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/133892/diff/2-3/

Comment 29

[Nevin Chen(Not active on Bugzilla)]

Comment on attachment 8861872 [details]
Bug 1329152 - turn on CustomTabs via SwitchBoard

https://reviewboard.mozilla.org/r/133892/#review187382

Comment 30

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e692d1d1be1e
Enable custom tabs by default r=sebastian
https://hg.mozilla.org/integration/autoland/rev/7033e71f7929
turn on CustomTabs via SwitchBoard r=nechen,sebastian

Comment 31

[Julian Chu [:walkingice]]

Got confirmation from :jcheng on slack, then I rebased the patches to land. After these patches, CustomTabs is enabled in nightly, and only in nightly so far.

Comment 32

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/e692d1d1be1e
https://hg.mozilla.org/mozilla-central/rev/7033e71f7929

Comment 33

[Nicole Yee (:nicoleyee)]

Hey Joe, Nevin and Wesly,

Did we turn this on with switchboard by default?

Comment 34

[Nevin Chen(Not active on Bugzilla)]

Nicole:
Curretnly we only have nightly with 100% 
Other channels are not set yet.

Julian:
I forgot about the logic. I remember it was "default off", we look into the switchboard and then update it accordingly, right?

Joe:
Maybe we need to turn it on for all channels now?

schema	1480618438089
name	"custom-tabs"
match	
appId	"^org.mozilla.fennec.*$"
buckets	
min	"0"
max	"100"
id	"5e23b482-8800-47be-b6dc-1a3bb6e455d4"
description	"Allows apps to open tabs in a customized UI."
last_modified	1493375228887

Comment 35

[Julian Chu [:walkingice]]

It is default off. CustomTabs won't be enabled until the user is in experiment via SwitchBoard. As your comment, now we only enable it in nightly.

Comment 36

[Joe Cheng [:jcheng] (please needinfo)]

let's turn this on 100% for beta.
Thanks