Port the Interventions experiment into a new provider
Categories
(Firefox :: Address Bar, enhancement, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox74 | --- | fixed |
People
(Reporter: bugzilla, Assigned: bugzilla)
References
(Blocks 2 open bugs)
Details
Attachments
(2 files)
|
||
Updated•5 years ago
|
|
Assignee | |
Comment 1•5 years ago
|
||
I think it will be easier to do this in one patch, rather than porting Interventions then adding the Update (bug 1606918) and Refresh (bug 1606919) interventions separately. I'm going to close those bugs and bump the points on this one.
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
||
Comment 5•5 years ago
|
||
| bugherder | ||
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
|
||
Comment 8•5 years ago
|
||
== Change summary for alert #24884 (as of Fri, 07 Feb 2020 08:36:02 GMT) ==
Improvements:
2% sessionrestore linux64-shippable opt e10s stylo 576.46 -> 562.42
2% startup_about_home_paint linux64-shippable opt e10s stylo 804.92 -> 787.58
For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=24884
|
||
Comment 9•4 years ago
|
||
Was there a particular reason to hardcode the sumo links here rather than using the baseURL method with Services.urlFormatter.formatURLPref to get version+locale appropriate SUMO redirects?
|
|
Assignee | |
Comment 10•4 years ago
|
||
There was not. Should I file a folllow-up, or has this come up in a patch you're already working on?
|
|
||
Comment 11•4 years ago
|
||
(In reply to Harry Twyford [:harry] from comment #10)
There was not. Should I file a folllow-up, or has this come up in a patch you're already working on?
It's coming up in bug 1693133 because we need a SUMO link there and I thought I'd use the same one that this code used, but then it turned out it wasn't using one. Normally we have to ask the SUMO folks to create a redirect if one doesn't exist, but I don't know if there is one already or not. If you would file a follow-up bug blocking this and bug 1693133 and needinfo Joni (jsavage@) to check what does and doesn't exist already, that would be great. I also don't know if we want different ones per feature to work out where traffic comes from... Again, hopefully they know what to do there...
|
|
Assignee | |
Updated•4 years ago
|
|
||
Comment 12•4 years ago
|
||
Sorry for not catching that in review.
Description
•