Closed
Bug 1181822
Opened 11 years ago
Closed 10 years ago
Add an A/B test to for low-res tiles vs. fading tiles
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(firefox49 fixed)
RESOLVED
FIXED
Firefox 49
| Tracking | Status | |
|---|---|---|
| firefox49 | --- | fixed |
People
(Reporter: snorp, Assigned: snorp)
References
Details
Attachments
(3 files, 1 obsolete file)
We could try only doing this in Nightly to start with. The reasoning here is:
1) We have a lot of issues with low-res tiles appearing when they shouldn't. The gfx folks are looking into this stuff now, but...
2) It seems like it may be more trouble than it's worth.
3) With bug 1178376 applied, which allows fading in new tiles, checkerboarding feels a little less jarring.
I propose we try the fading tiles stuff on Nightly to see how folks like it.
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8631306 -
Flags: review?(mark.finkle)
Attachment #8631306 -
Flags: review?(blassey.bugs)
Attachment #8631306 -
Flags: feedback?(alam)
Comment 3•11 years ago
|
||
I suspect you'll want a video or a build. Screenshot might not be that interesting.
Comment 4•11 years ago
|
||
I'm testing on CNN and the low-res tiles are much better
Updated•11 years ago
|
Attachment #8631306 -
Flags: review?(blassey.bugs) → review-
| Assignee | ||
Comment 5•11 years ago
|
||
Here's a build with this enabled: http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/jwillcox@mozilla.com-cb0e9a97ce2b/try-android-api-11/fennec-42.0a1.en-US.android-arm.apk
Flags: needinfo?(snorp) → needinfo?(alam)
Comment 6•11 years ago
|
||
Tested this on my N6 and if I'm understanding this correctly, I like the fading in more.
Even if through measurement, loading the lower res tiles is slightly faster, I think there's something to be said about perceived performance here. It even feels smoother to me.
Flags: needinfo?(alam)
Comment 7•11 years ago
|
||
On single core devices (Nexus S Android 4.0) this just does not work. The tiles don't fade in just pop.
Updated•11 years ago
|
Attachment #8631306 -
Flags: feedback?(alam)
| Assignee | ||
Comment 8•10 years ago
|
||
| Assignee | ||
Comment 9•10 years ago
|
||
| Assignee | ||
Updated•10 years ago
|
Assignee: nobody → snorp
| Assignee | ||
Comment 10•10 years ago
|
||
Ignore the weird urlbar issue on the fading one, that seems to be some regression in Nightly when using synthetic input events.
| Assignee | ||
Comment 11•10 years ago
|
||
We want to use an A/B test to determine if replacing the low-res tiles with fading ones will improve the experience.
Summary: Disable low-res tiles, enable fade-in of high-res tiles → Add an A/B test to for low-res tiles vs. fading tiles
| Assignee | ||
Comment 12•10 years ago
|
||
Attachment #8631306 -
Attachment is obsolete: true
Attachment #8631306 -
Flags: review?(mark.finkle)
Attachment #8659942 -
Flags: feedback?(mark.finkle)
Comment 13•10 years ago
|
||
Comment on attachment 8659942 [details] [diff] [review]
Add A/B experiment for low-res tiles vs. fading on Android
This looks like a good approach. I would use ExperimentHelper for Gecko-based pref experiments. Most Java/UI experiments will have the code in to UI code itself. Not to say we might not find a bigger use for ExperimentHelper.
I would rename runABExperiments() -> setup()
This is a good start.
If you add some NIGHTLY_BUILD support in there, I can add the segmentation to the switchboard server and we can make this live.
Attachment #8659942 -
Flags: feedback?(mark.finkle) → feedback+
Comment 14•10 years ago
|
||
Another thing we should think about is the metrics for the experiment. What can we measure to see if we get a positive improvement. We are gearing up to get churn metrics, but those can be difficult to correlate back to a specific experiment. We'd like some simple data we can send to telemetry.
Comment 15•10 years ago
|
||
In the future, we could even do further this experiment by looking at more than just fading vs low res.
From convos with snorp on IRC, for example, we could test different ways to optimize perceived perforance through changes in fading duration, linear vs non-linear fading tweens, etc.
| Assignee | ||
Comment 16•10 years ago
|
||
I've revived the code that makes fading tiles work, and I would like to turn it on by default in Nightly only. I am not sure what benefit we would really get from a switchboard experiment, and I would like to get the most testing possible with the fading tiles. Anthony does this sound ok to you?
Flags: needinfo?(alam)
Comment 17•10 years ago
|
||
(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) from comment #16)
> I've revived the code that makes fading tiles work, and I would like to turn
> it on by default in Nightly only. I am not sure what benefit we would really
> get from a switchboard experiment, and I would like to get the most testing
> possible with the fading tiles. Anthony does this sound ok to you?
Sounds fine by me. Let's make sure we're letting everyone else know too (like when we turned on gecko carets for Nightly users) so we can get more feedback :D
Flags: needinfo?(alam) → needinfo?(bbermes)
Comment 18•10 years ago
|
||
Comment 19•10 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Comment 20•10 years ago
|
||
To be honest, as per Anthony's comment in 6, I'm having issues being persued that this is better performance.
It's been more "annoying" and disturbing to me to get the black screen fading in and out and stopping me from "thinking" I'm continuing my browsing experience. Obviously being subjective here, and I'd like to get more users giving feedback.
Flags: needinfo?(bbermes)
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•