app splash screen icon makes it feel generally slow

VERIFIED FIXED in Firefox 56

Status

()

Firefox for Android
Theme and Visual Design
VERIFIED FIXED
a year ago
10 months ago

People

(Reporter: dietrich, Assigned: Nevin Chen(Not active on Bugzilla))

Tracking

(Blocks: 1 bug, {perf, regression})

unspecified
Firefox 56
perf, regression
Points:
---

Firefox Tracking Flags

(fennec+, firefox54 unaffected, firefox55 unaffected, firefox56 verified, firefox57 verified)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

a year ago
The app used to load the UI and then the web page, which gave the impression it was busy loading the web page.

Now it loads the splash screen with the Firefox logo as the app icon, which I see regularly.

Overall, this gives the feeling of a much slower app. Instead of attempting to load the web page, it appears it's now struggling to just load itself.

Adding "regression" since this is a UX regression in the area of perceived performance.
Blocks: 1366674
Maybe it might feel better if most of the splash screen (save for example a small circular area immediately surrounding the logo) was transparent, so you could still see the actual UI starting up and pretending to load the page (even if it is in fact still starting up Gecko). But generally I agree - the current splash screen implementation just makes things feel slower.
tracking-fennec: --- → ?
This change also increased the size of the fennec apk by about 350k:

== Change summary for alert #7808 (as of July 11 2017 03:57 UTC) ==

Regressions:

  1%  installer size summary android-4-0-armv7-api15 opt      36,200,098.58 -> 36,544,662.00

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=7808
(Assignee)

Updated

11 months ago
No longer blocks: 1366674
(Assignee)

Comment 3

11 months ago
sorry all splash screen bug all block bug 1379657 directly so I remove blocking bug 1366674.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
I just filed 1380008 for the APK size issue, thanks wlach!
(Assignee)

Updated

11 months ago
Assignee: nobody → cnevinchen

Comment 7

11 months ago
mozreview-review
Comment on attachment 8885172 [details]
Bug 1378688 - Make Splash Screen faster by making it's background transparent.

https://reviewboard.mozilla.org/r/156034/#review161514
Attachment #8885172 - Flags: review?(max) → review+
(Reporter)

Comment 8

11 months ago
I'm concerned that showing any kind of splash icon will reduce perceived performance. We want to give the impression that Firefox is always ready instantly, even if the web page is not.

Does this new approach result in the application UI being fully visible? That will give the feeling that Firefox is always snappy and responsive.
(Reporter)

Comment 9

11 months ago
Actually, if we're showing the application UI fully rendered, then why show the application icon at all? Instead could show a loading animation in the viewport. I think that would result in better perceived performance, and tells a more truthful story about all the various reasons things might be slow in loading/rendering the *web page*, as distinct from the *application*.
(Reporter)

Comment 10

11 months ago
Another interesting optimization could be to show a loading animation and then once the favicon of the website is available, show that in the animation somehow. That tells an even clearer story about what's being slow.

Updated

11 months ago
tracking-fennec: ? → +

Comment 11

11 months ago
[triage@0712] Nevin is working on it. +
(Assignee)

Comment 12

11 months ago
Hi Dietrich!
Thanks for your feedback!

I've submit a new version.
Please help check in Nightly after it lands:)
(Assignee)

Updated

11 months ago
Duplicate of this bug: 1378175
Comment hidden (mozreview-request)
(Assignee)

Updated

11 months ago
Duplicate of this bug: 1380008
(Assignee)

Updated

11 months ago
Duplicate of this bug: 1376579
We need to be careful about loading animations. In the past we've seen that those can be very costly in terms of battery life, and in some cases they can slow down the actual page load. I'll see if I can dig up some bugs from years back.

Comment 19

11 months ago
mozreview-review
Comment on attachment 8885652 [details]
Bug 1378688 - Scale the image to save APK size, respect channel, and hide splash screen when the user interact with url bar.

https://reviewboard.mozilla.org/r/156474/#review161894

::: mobile/android/base/java/org/mozilla/gecko/widget/SplashScreen.java:7
(Diff revision 1)
> +import android.util.Log;
> +import android.view.View;

Not secessary import?
Attachment #8885652 - Flags: review?(max) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 25

11 months ago
Hi snorp
I totaly agree with you about the CPU usage. And even memory consumtion( We scale the logo image to reduce the APK size). This will increase ~8MB memory footprint when startup. But at this time I think the hardware spec is much better now. I'll still want to see how the users repsonse to our design for "perceived performance".
Flags: needinfo?(snorp)
Comment hidden (mozreview-request)

Comment 27

11 months ago
Pushed by nechen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/29ab214f96d2
Make Splash Screen faster by making it's background transparent. r=maliu
https://hg.mozilla.org/integration/autoland/rev/7b5d4791c970
Scale the image to save APK size, respect channel, and hide splash screen when the user interact with url bar. r=maliu
(Reporter)

Comment 28

11 months ago
(In reply to Nevin Chen [:nechen] from comment #25)
> I'll still want to see how the users repsonse to our design for
> "perceived performance".

Hey Nevin, do you have any background on why we added the splash screen to begin with?

I'm pretty skeptical that showing a Firefox logo will improve perceived performance in any way.

I think it does the opposite. It tells a clear story about how *Firefox* is slow, instead of how the *network or website* is slow.
(Reporter)

Comment 29

11 months ago
I'm curious, maybe Snorp you know this: Do we have telemetry around how often we get launched from app icon vs link click?

I filed this bug because my use of the browser feels almost entirely from app-to-app, not homescreen launches, which made the splash screen feel especially intrusive in the workflow.

I can understand the Firefox logo splash on launch from homescreen. But when loading from other app, seems like we want to do everything possible to instantly launch app UI and push any perceived slowness away onto other potential causes.

Comment 30

11 months ago
AFAIK the splash screen is just a resource and thus is displayed while the system is trying to load the UI. However Fennec takes longer than your average Android app to load the UI due to using custom UI elements. I'm not sure if that'll be negated after Photon.
(In reply to Paul [pwd] from comment #30)
> AFAIK the splash screen is just a resource and thus is displayed while the
> system is trying to load the UI. However Fennec takes longer than your
> average Android app to load the UI due to using custom UI elements.

While Gecko does unfortunately indeed need a few seconds to get up and running, the rest of our UI actually appears fairly quickly. There's only a short flash of white (which by the way is still there even with the splash screen) before we show the address bar (complete with the progress bar pretending we're already loading the page, even if in fact we're still busy starting up Gecko first), and once that happens you can already interact with the app in some ways, e.g. open/close tabs, look at the awesome screen, paste a link to open it, etc.
Hence the splash screen (especially the current variety on Nightly with the all white background) makes things feel slower, because now it seems like even our UI is taking several seconds to start up.
(In reply to Dietrich Ayala (:dietrich) from comment #29)
> I'm curious, maybe Snorp you know this: Do we have telemetry around how
> often we get launched from app icon vs link click?

We do record telemetry on this[1], but I don't really know how to find the results.

https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java#4293
Flags: needinfo?(snorp)

Comment 33

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/29ab214f96d2
https://hg.mozilla.org/mozilla-central/rev/7b5d4791c970
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
(Reporter)

Comment 34

11 months ago
Filed a follow-up to re-examine the logo issue; bug 1380698
status-firefox54: --- → unaffected
status-firefox55: --- → unaffected

Comment 35

10 months ago
Verified that the URL bar and loading indicator are now visible on Nightly 57 and Beta 56 and the screen closes if you interact with the URL bar.
Status: RESOLVED → VERIFIED
status-firefox56: fixed → verified
status-firefox57: --- → verified
QA Contact: oana.horvath
You need to log in before you can comment on or make changes to this bug.