crash in java.lang.NullPointerException: at android.support.design.widget.Snackbar.<init>(Unknown Source)

RESOLVED FIXED in Firefox 45

Status

()

Firefox for Android
Web Apps
--
critical
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kats, Assigned: sebastian)

Tracking

({crash})

45 Branch
Firefox 46
Unspecified
Android
crash
Points:
---

Firefox Tracking Flags

(firefox45 fixed, firefox46 fixed, fennec45+)

Details

(crash signature)

MozReview Requests

()

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

Attachments

(1 attachment)

This bug was filed from the Socorro interface and is 
report bp-37b4cd90-bf6c-4da6-84cd-a59942160102.
=============================================================

Install a webapp using aurora. Open it, long-press on a link, and select copy link. I haven't tried reproducing it yet.
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
tracking-fennec: --- → ?
status-firefox45: --- → affected
status-firefox46: --- → affected
Created attachment 8705726 [details]
MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella

Review commit: https://reviewboard.mozilla.org/r/30173/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/30173/
Attachment #8705726 - Flags: review?(michael.l.comella)
(In reply to Sebastian Kaspari (:sebastian) from comment #2)
> Created attachment 8705726 [details]
> MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if
> it exists (Webapps). r?mcomella
> 
> Review commit: https://reviewboard.mozilla.org/r/30173/diff/#index_header
> See other reviews: https://reviewboard.mozilla.org/r/30173/

This patch is built on top of the changes in bug 1234295.
Comment on attachment 8705726 [details]
MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella

https://reviewboard.mozilla.org/r/30173/#review27187

::: mobile/android/base/java/org/mozilla/gecko/SnackbarHelper.java:135
(Diff revision 1)
> -            return activity.findViewById(R.id.root_layout);
> +            View view = activity.findViewById(R.id.root_layout);

nit: `final`
Attachment #8705726 - Flags: review?(michael.l.comella) → review+
Attachment #8705726 - Attachment description: MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r?mcomella → MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella
Comment on attachment 8705726 [details]
MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/30173/diff/1-2/
Comment on attachment 8705726 [details]
MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella

(Note: This patch is build on top of the patches in bug 1234295. I requested uplift for those patches too.)

Approval Request Comment

[Feature/regressing bug #]: Snackbars have been introduced app-wide in Firefox 45 - see meta bug 1157526.

[User impact if declined]: Whenever we show a snackbar in a webapp the app might crash.

[Describe test coverage new/current, TreeHerder]: Local testing and try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d3b88a6cf94a

[Risks and why]: Low risk: Very minimal patch.

[String/UUID change made/needed]: -
Attachment #8705726 - Flags: approval-mozilla-aurora?

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/92e652055276
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox46: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment on attachment 8705726 [details]
MozReview Request: Bug 1236232 - Only use root layout as snackbar parent if it exists (Webapps). r=mcomella

Taking in 45. Thanks
Attachment #8705726 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
tracking-fennec: ? → 45+

Comment 11

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/b32e085b9f62
status-firefox45: affected → fixed
You need to log in before you can comment on or make changes to this bug.