1482478 - PROCESS-CRASH | dom/media/tests/mochitest/test_getUserMedia_cubebDisabled.html | java-exception java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference at org.mozilla.

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Download Manager, defect)

Description

[Bob Clary [:bc] (inactive)]

bug 1476966 introduced a permacrash for mda3 on Pixel2 hardware https://hg.mozilla.org/integration/autoland/rev/925d2fd06a2451ceee7c9136fe6f337a87c2af67

Note this a tier 3 job.

https://treeherder.mozilla.org/#/jobs?repo=autoland&filter-searchStr=android-hw&filter-tier=1&filter-tier=2&filter-tier=3&fromchange=65a6a118cb1a7bbe4c2af3ee41e89eb6704f48bc&selectedJob=193235762

PROCESS-CRASH | dom/media/tests/mochitest/test_getUserMedia_cubebDisabled.html | java-exception java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference at org.mozilla.gecko.notifications.NotificationHelper.showNotification(NotificationHelper.java:356)

Andrei: Please fix or backout otherwise I'll have to spend time starring these every time they fail.

Comment 1

[Bob Clary [:bc] (inactive)]

Maybe a dupe of bug 1481957 ?

Comment 2

[Intermittent Failures Robot]

28 failures in 502 pushes (0.056 failures/push) were associated with this bug yesterday.

Repository breakdown:
* mozilla-central: 4
* mozilla-inbound: 4
* autoland: 20

Platform breakdown:
* android-hw-p2-8-0-arm7-api-16: 28

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1482478&startday=2018-08-10&endday=2018-08-10&tree=all

Comment 5

[Stefan Deiac]

Last good revision: 925d2fd06a2451ceee7c9136fe6f337a87c2af67
First bad revision: d05ae84b507f7c0ba986f09202d8be8b3b2eded2
Pushlog:https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=925d2fd06a2451ceee7c9136fe6f337a87c2af67&tochange=d05ae84b507f7c0ba986f09202d8be8b3b2eded2

Comment 6

[Calixte Denizet (:calixte)]

Stefan, why did you remove regression keyword ?

Comment 7

[Stefan Deiac]

Sorry for mistake, was a misunderstanding of the flag.

Comment 8

[Intermittent Failures Robot]

124 failures in 2579 pushes (0.048 failures/push) were associated with this bug in the last 7 days.

This is the #7 most frequent failure this week.

** This failure happened more than 75 times this week! Resolving this bug is a very high priority. ** 

** Try to resolve this bug as soon as possible. If unresolved for 1 week, the affected test(s) may be disabled. ** 

Repository breakdown:
* autoland: 56
* mozilla-central: 28
* mozilla-inbound: 40

Platform breakdown:
* android-hw-p2-8-0-arm7-api-16: 124

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1482478&startday=2018-08-06&endday=2018-08-12&tree=all

Comment 9

[Daniel Holbert [:dholbert]]

I can reliably hit this in Nightly on my Pixel 2 by doing anything that triggers an "allow-mic-access" prompt.

Sample STR:
1. Visit https://appear.in/testmoz (i.e. attempt to join this video/audio chat room named "testmoz"
2. Accept the prompt to allow mic/camera access.

ACTUAL RESULTS: Instant crash.

Comment 10

[Daniel Holbert [:dholbert]]

(In reply to Bob Clary [:bc:] from comment #1)
> Maybe a dupe of bug 1481957 ?

It looks like my Android Nightly doesn't have that bug's patch yet (built from https://hg.mozilla.org/mozilla-central/rev/bf79440c1376b1e1114ba653917e1577d7b1007b which hit central a handful of hours earlier than that bug).  I anticipate the next Nightly will have that bug's fix, and I'll report back once I've got it.

Comment 11

[Jan Henning [:JanH]]

Attachment: Bug 1482478 - Treat handler attribute in notification message as optional. r?jchen

Comment 12

[Bob Clary [:bc] (inactive)]

fixed by bug 1481957

Comment 13

[Jan Henning [:JanH]]

Looking at the rest of the code, the "handler" attribute is optional, so I still think we should fix this in NotificationHelper.java as well.

Comment 14

[(inactive) Jim Chen [:jchen] [:darchons]]

Comment on attachment 9000052 [details]
Bug 1482478 - Treat handler attribute in notification message as optional. r?jchen

Jim Chen [:jchen] [:darchons] has approved the revision.

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cfb5c70e0ed
Treat handler attribute in notification message as optional. r=jchen

Comment 16

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/8cfb5c70e0ed

Comment 17

[Intermittent Failures Robot]

32 failures in 2672 pushes (0.012 failures/push) were associated with this bug in the last 7 days.

** This test has failed more than 150 times in the last 21 days. It should be disabled until it can be fixed. ** 

Repository breakdown:
* mozilla-central: 4
* autoland: 10
* mozilla-inbound: 18

Platform breakdown:
* android-hw-p2-8-0-arm7-api-16: 32

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1482478&startday=2018-08-13&endday=2018-08-19&tree=all