Closed Bug 1758212 Opened 2 years ago Closed 2 years ago

GeckoView crashes at startup without a ContentDelegate

Categories

(GeckoView :: General, defect)

Unspecified
All
defect

Tracking

(firefox-esr91 unaffected, firefox97 wontfix, firefox98 wontfix, firefox99 wontfix, firefox100 fixed)

RESOLVED FIXED
100 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox97 --- wontfix
firefox98 --- wontfix
firefox99 --- wontfix
firefox100 --- fixed

People

(Reporter: agi, Assigned: m_kato)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Noticed while following the getting started doc, in here https://searchfox.org/mozilla-central/rev/2bd64785fc470d8484df9cb4a4e03e50b3dc3060/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoSession.java#6197

we assume that we have a ContentDelegate, while that might not be the case. We need to add a null check there.

Regressed by: 1672609

Set release status flags based on info from the regressing bug 1672609

:m_kato, since you are the author of the regressor, bug 1672609, could you take a look?
For more information, please visit auto_nag documentation.

Flags: needinfo?(m_kato)
Has Regression Range: --- → yes
Assignee: nobody → m_kato
Flags: needinfo?(m_kato)

Set release status flags based on info from the regressing bug 1672609

Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/4b62b4efe0c3
Add null check before calling ContentDelegate.onPointerIconChange. r=geckoview-reviewers,agi
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: