Closed Bug 1403980 Opened 2 years ago Closed 2 years ago

Splash screen displayed in new tab opened from the search widget

Categories

(Firefox for Android :: Theme and Visual Design, defect)

ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 58
Tracking Status
fennec + ---
firefox56 --- wontfix
firefox57 --- wontfix
firefox58 --- verified

People

(Reporter: oana.horvath, Assigned: cnevinchen)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [FNC][SPT58.2][MVP])

Attachments

(2 files)

Build: Nightly 58.0a1 (2017-09-28);

Steps to reproduce:
1. Load a page in Fennec.
2. Close the Fennec app.
3. Tap the new tab button from the search widget.
4. While Fennec is opening a new tab, verify the splash screen.

Expected result:
Based on Bug 1366674, the splash screen should be shown only if there is a non-empty tab loading at startup.

Actual result:
The splash screen is displayed while opening an empty tab.
Flags: needinfo?(chuang)
yes, it's not supposed to show the splash screen here.

Hi Joe,
Could we put this on our list and see the priority of this bug?
thanks!
Flags: needinfo?(chuang) → needinfo?(jcheng)
we should fix this if it is 100% reproducible
does it impact 56,57,58?
Flags: needinfo?(wehuang)
Flags: needinfo?(vchen)
Flags: needinfo?(topwu.tw)
Flags: needinfo?(jcheng)
Flags: needinfo?(cnevinchen)
Yes, it's 100% reproducible and affects all 3: 56,57,58. Thanks!
Assignee: nobody → cnevinchen
Flags: needinfo?(cnevinchen)
Comment on attachment 8914006 [details]
Bug 1403980 - Hide Splash screen in new tab opened from the search widget.

https://reviewboard.mozilla.org/r/185402/#review190364
Attachment #8914006 - Flags: review?(topwu.tw) → review+
Flags: needinfo?(topwu.tw)
When we open Firefox through search widget, we restore the previous pages first[1] and then open a new tab for the search wdiget.
There's no way we can know the search widget will open a new tab (with about:home) later when [1] happen.

So we'll still show splash screen here for search widget with about:home as "new tab"

Hi Carol
Do you need a video for this?
Flags: needinfo?(chuang)
Hi Nevin, 
According to our offline discussion, i think the solution works. Showing the splash screen before the new tab (without overlapping) if the user open the app for the first time. Thanks!
Flags: needinfo?(chuang)
Pushed by nechen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d44bbce04733
Hide Splash screen in new tab opened from the search widget. r=jwu
https://hg.mozilla.org/mozilla-central/rev/d44bbce04733
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Flags: needinfo?(wehuang)
Verified as fixed in build 58.0a1 (2017-10-11);
Devices: Google Pixel (Android 8.0) and Sony Xperia Z5 Premium (Android 6.0.1).
Whiteboard: [FNC][SPT58.2][MVP]
Let's uplift to 57 if we still can. But even if we don't uplift, it's minimal impact as it only happens with search widget, which I assume the usage is low when Google search widget is on most phones by default so we should not block 57 with this.
Flags: needinfo?(vchen) → needinfo?(cnevinchen)
Hey Joe,

what is the impact to users for this bug? Unfortunately, we are very late in 57 beta cycle and threshold for uplifts is extremely high. The guideline that was given to me is if its not dot release worthy, it will not be in 57. But if impact to users is high enough, we can plead our case.
Per discussion in Bug 1405853#c11[1] I thought we can still uplift this before Monday? I know for Desktop it's basically in "regression/crash only" mode but maybe not exactly the same case for Mobile?

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1405853#c11
Comment on attachment 8914006 [details]
Bug 1403980 - Hide Splash screen in new tab opened from the search widget.

Approval Request Comment
[Feature/Bug causing the regression]: Splash Screen will apears when users click "New Tab" from search widget. Browser will show home panel and splash screen at the same time.
[User impact if declined]: UI is not good
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]:  no 
[List of other uplifts needed for the feature/fix]: no 
[Is the change risky?]:no 
[Why is the change risky/not risky?]: just changing ui
[String changes made/needed]:no
Flags: needinfo?(cnevinchen)
Attachment #8914006 - Flags: approval-mozilla-beta?
Comment on attachment 8914006 [details]
Bug 1403980 - Hide Splash screen in new tab opened from the search widget.

This doesn't seem like a must fix. I'd prefer this change ride the 58 train.
Attachment #8914006 - Flags: approval-mozilla-beta? → approval-mozilla-beta-
tracking-fennec: ? → +
You need to log in before you can comment on or make changes to this bug.