Closed Bug 1271698 Opened 4 years ago Closed 4 years ago
Put recent URL bar changes behind Switchboard flags
58 bytes, text/x-review-board-request
67 bytes, text/x-github-pull-request
|Details | Review|
5.18 KB, patch
|Details | Diff | Splinter Review|
5.33 KB, patch
|Details | Diff | Splinter Review|
We want to put behind Switchboard flags: * Showing origin only in URL bar (bug 1236431) * Show name of organization (EV certificate) (bug 1249594)
* urlbar-show-origin-only: Only show origin in URL bar instead of full URL (Bug 1236431) * urlbar-show-ev-cert-owner: Show name of organization (EV cert) instead of full URL in URL bar (Bug 1249594) Review commit: https://reviewboard.mozilla.org/r/51911/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/51911/
Attachment #8751220 - Flags: review?(margaret.leibovic)
For now the experiment is only enabled on Nightly, Aurora and Beta.
It might be good to create some guidelines for creating experiments vs staging rollouts. In this case, do we need a different experiment for each variation? Or could we use a single experiment ("urlbar-show") and have sub-variables for the other state?
(In reply to Mark Finkle (:mfinkle) from comment #4) > It might be good to create some guidelines for creating experiments vs > staging rollouts. In this case, do we need a different experiment for each > variation? Or could we use a single experiment ("urlbar-show") and have > sub-variables for the other state? We could use experiment values for enabling the sub-parts of the feature. However, we will have less control if that's the case, since match/buckets would only apply to the entire experiment. For example, we couldn't turn on only the origin part for release, but keep the EV cert owner enabled up to beta. What problems do you identify with different experiments for each variation? I understand it's more things to keep track of, and can lead to a long list of experiments, but I like that it is more explicit.
At first I wanted to create just one experiment but then I created two. Just because I do not want to get back to it if we decide that we want to show the origin but want to revert the EV certificate changes.
Comment on attachment 8751220 [details] MozReview Request: Bug 1271698 - Add Switchboard flags for URL bar changes. r?margaret https://reviewboard.mozilla.org/r/51911/#review48751
Attachment #8751220 - Flags: review?(margaret.leibovic) → review+
Comment on attachment 8751223 [details] [review] (GitHub) Switchboard Config: Bug 1271698 - Add experiments for URL bar changes. This doesn't require relman approval, since it won't actually change anything until your patch is uplifted. So feel free to merge whenever.
Attachment #8751223 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/integration/fx-team/rev/dfb23d172953e063e321c20feec282e955230310 Bug 1271698 - Add Switchboard flags for URL bar changes. r=margaret
Comment on attachment 8751220 [details] MozReview Request: Bug 1271698 - Add Switchboard flags for URL bar changes. r?margaret Approval Request Comment [Feature/regressing bug #]: We want to put the changes from bug 1236431 and bug 1249594 behind switchboard flags to control the release of this feature while we evaluate the user feedback we received. The switchboard configuration keeps the features enabled on Nightly, Aurora and Beta. The features are not in release yet but after the merge day we can use switchboard to decide if we want to roll this out or not. [User impact if declined]: Without that the features will ride the trains to release after the next merge day and we might not have enough time left to make changes based on the feedback. [Describe test coverage new/current, TreeHerder]: Local testing. [Risks and why]: Low - This patch just adds the switchboard experiments and falls back to the default behavior for URLs if the user is in no experiment. [String/UUID change made/needed]: -
Comment on attachment 8751220 [details] MozReview Request: Bug 1271698 - Add Switchboard flags for URL bar changes. r?margaret Switchboard flags for dynamic control of feature per channel. Aurora48+, Beta47+
Hitting conflicts uplifting to aurora.
Modified patch for Aurora.
Modified patch for Beta.
Hi Wes, just fyi, the rebased patches are ready to land on m-b. Thanks!
Verified as fixed in build: - Aurora 48.0a2 (2016-05-26); - Nightly 49.0a1 (2016-05-26); Device: LG G4 (Android 5.1)
You need to log in before you can comment on or make changes to this bug.