Closed Bug 1407915 Opened 5 years ago Closed 5 years ago

Crash in java.lang.IllegalStateException: Page URL is required at


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

58 Branch
Not set


(firefox-esr52 unaffected, firefox56 unaffected, firefox57 fixed, firefox58 fixed)

Firefox 58
Tracking Status
firefox-esr52 --- unaffected
firefox56 --- unaffected
firefox57 --- fixed
firefox58 --- fixed


(Reporter: calixte, Assigned: jwu)


(Blocks 1 open bug)


(Keywords: crash, regression, topcrash, Whiteboard: [clouseau][FNC][SPT58.2][INT])

Crash Data


(1 file)

This bug was filed from the Socorro interface and is 
report bp-0a44a468-32eb-4fca-ad29-06b160171012.

There are 140 crashes in nightly 58 with buildid 20171011100102. In analyzing the backtrace, the regression may have been introduced by patch [1] to fix bug 1406338.
The crash is ranked #2 in nightly top-crashers for FennecAndroid.

Flags: needinfo?(
We should prevent loading favicon where the request URL is empty.
Assignee: nobody →
Flags: needinfo?(
Comment on attachment 8917713 [details]
Bug 1407915 - Ignore loading favicon without url.
Attachment #8917713 - Flags: review?(cnevinchen) → review+
Pushed by
Ignore loading favicon without url. r=nechen
Comment on attachment 8917713 [details]
Bug 1407915 - Ignore loading favicon without url.

Approval Request Comment
[Feature/Bug causing the regression]:
A regression caused by bug 1406338, we should prevent loading favicon when there is no valid URL.

[User impact if declined]:
If there is a tab contained empty URL, Fennec would crash when user enters tabs tray page.

[Is this code covered by automated tests?]:

[Has the fix been verified in Nightly?]:
Not yet

[Needs manual test from QE? If yes, steps to reproduce]: 
Yes, here is the STR:

Prerequisite: Create a new tab with empty URL, however, I don't know how to create a tab without URL, perhaps you have to test several times.

1. Launch Fennec and press the 'tabs' button to enter tabs tray page,
2. Check if the favicon for each tab is shown correctly and app doesn't crash.

[List of other uplifts needed for the feature/fix]:

[Is the change risky?]:

[Why is the change risky/not risky?]:
We just add an error checking to prevent loading favicon if we get an empty URL.

[String changes made/needed]:
Attachment #8917713 - Flags: approval-mozilla-beta?
There seem to be many more crashes on 58 than on 57 - I see only one crash on 57 beta.
Comment on attachment 8917713 [details]
Bug 1407915 - Ignore loading favicon without url.

Low volume, low risk, new crash, Beta57+
Attachment #8917713 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Whiteboard: [clouseau] → [clouseau][FNC][SPT58.2][INT]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.