Setting a homepage makes Nightly unresponsive

VERIFIED FIXED in Firefox 47

Status

()

defect
--
major
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: TeoVermesan, Assigned: sebastian)

Tracking

({crash, regression})

48 Branch
Firefox 48
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox46 unaffected, firefox47+ verified, firefox48 verified, fennec47+)

Details

Attachments

(3 attachments)

Steps to reproduce:
1. Menu -> Settings -> General -> Home-> Set a homepage
2. Choose "Custom" and enter the address or search term:"www.google.com" and tap ok
3. Close Firefox and reopen

Expected results:
- A tab is opened with "www.google.com"

Actual results:
- Nightly crashes, but I cannot see the crash report because after restart a blank page is opened
- Nightly becomes unresponsive and cannot be used anymore
- If I "Force Stop" from Android Settings and tap the Nightly icon it opens for a second and closes.

Regression:
08-03 good build
09-03 affected build

pushlog: 
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=05c087337043dd8e71cc27bdb5b9d55fd00aaa26&tochange=af7c0cb0798f5425d5d344cbaf0ac0ecb1a72a86
Summary: Settings a homepage makes Nightly unresponsive → Setting a homepage makes Nightly unresponsive
tracking-fennec: --- → ?
Can you attach a log?
Flags: needinfo?(teodora.vermesan)
Posted file homepagecrash.txt
Here are the attached logs while restarting Nightly and it crashes.
Flags: needinfo?(teodora.vermesan)
I see this in the log:

E/GeckoCrashHandler( 4167): java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.String.indexOf(java.lang.String)' on a null object reference
E/GeckoCrashHandler( 4167): 	at org.mozilla.gecko.toolbar.ToolbarDisplayLayout.updateFromTab(ToolbarDisplayLayout.java:213)
E/GeckoCrashHandler( 4167): 	at org.mozilla.gecko.toolbar.BrowserToolbar.onTabChanged(BrowserToolbar.java:496)
E/GeckoCrashHandler( 4167): 	at org.mozilla.gecko.Tabs$5.run(Tabs.java:674)
E/GeckoCrashHandler( 4167): 	at android.os.Handler.handleCallback(Handler.java:739)
E/GeckoCrashHandler( 4167): 	at android.os.Handler.dispatchMessage(Handler.java:95)
E/GeckoCrashHandler( 4167): 	at android.os.Looper.loop(Looper.java:135)
E/GeckoCrashHandler( 4167): 	at android.app.ActivityThread.main(ActivityThread.java:5254)
E/GeckoCrashHandler( 4167): 	at java.lang.reflect.Method.invoke(Native Method)
E/GeckoCrashHandler( 4167): 	at java.lang.reflect.Method.invoke(Method.java:372)
E/GeckoCrashHandler( 4167): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:903)
E/GeckoCrashHandler( 4167): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)

Mike, Sebastian, I see your names all over this regression window. Any ideas?
Flags: needinfo?(s.kaspari)
Flags: needinfo?(michael.l.comella)
I touched ToolbarDisplayLayout for the various URL bar updates. I'll have a look.
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Flags: needinfo?(s.kaspari)
Flags: needinfo?(michael.l.comella)
See Also: → 1255970
Duplicate of this bug: 1255970
Attachment #8730144 - Flags: review?(margaret.leibovic) → review+
Comment on attachment 8730144 [details]
MozReview Request: Bug 1255767 - ToolbarDisplayLayout: Only use baseDomain if it is available. r?margaret.

https://reviewboard.mozilla.org/r/39691/#review36367
https://hg.mozilla.org/integration/fx-team/rev/53961d91a52be96a4f5b2d091206c3bfea3a680b
Bug 1255767 - ToolbarDisplayLayout: Only use baseDomain if it is available. r=margaret.
https://hg.mozilla.org/mozilla-central/rev/53961d91a52b
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Set a homepage, then close Firefox and reopen it. A new tab with that homepage will be displayed:
Verified as fixed using:
Device: One A2001 (Android 5.1.1) 
Build: Firefox for Android 48.0a1 (2016-03-15)
Status: RESOLVED → VERIFIED
tracking-fennec: ? → 48+
[Tracking Requested - why for this release]: This needs to be uplifted to 47 to match bug 1250671
tracking-fennec: 48+ → 47+
Uplift to 47 please
Flags: needinfo?(s.kaspari)
It'd be great to add a regression test here: I filed bug 1258575.
Comment on attachment 8730144 [details]
MozReview Request: Bug 1255767 - ToolbarDisplayLayout: Only use baseDomain if it is available. r?margaret.

Approval Request Comment

[Feature/regressing bug #]: Regression introduced with uplift of bug 1250671.

[User impact if declined]: App crashes if homepage is set and app is started.

[Describe test coverage new/current, TreeHerder]: Local testing and this has been in Nightly for a week.

[Risks and why]: Low - This patch adds an additional null check that got removed with the URL bar behavior changes. It's in Nightly for a week now and there are no known regressions.

[String/UUID change made/needed]: -
Flags: needinfo?(s.kaspari)
Attachment #8730144 - Flags: approval-mozilla-aurora?
Tracked for Fx47, adding crash in keyword tagging.
Keywords: crash
Comment on attachment 8730144 [details]
MozReview Request: Bug 1255767 - ToolbarDisplayLayout: Only use baseDomain if it is available. r?margaret.

Crash fix that was verified and has been in Nightly 48 for a week, taking it in Aurora47.
Attachment #8730144 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Duplicate of this bug: 1258523
Set a homepage, then close Firefox and reopen it. A new tab with that homepage will be displayed:
Verified as fixed using:
Device: One A2001 (Android 5.1.1) 
Build: Firefox for Android 47 Beta 1
You need to log in before you can comment on or make changes to this bug.