1375749 - Intermittent /service-workers/service-worker/xhr.https.html | application crashed [@ RefPtr<mozilla::dom::FetchBody<mozilla::dom::Response> >::operator->]

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, defect)

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

https://treeherder.mozilla.org/logviewer.html#?job_id=109345003&repo=mozilla-central

https://queue.taskcluster.net/v1/task/IaRwGQWKQ7u0JGzbCSjrWA/runs/0/artifacts/public/logs/live_backing.log

Comment 2

[Intermittent Failures Robot]

3 failures in 892 pushes (0.003 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* autoland: 2
* mozilla-central: 1

Platform breakdown:
* linux64: 2
* windows7-32-vm: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1375749&startday=2017-06-19&endday=2017-06-25&tree=all

Comment 3

[Andrea Marchesini [:baku]]

Attachment: crash.patch

I was able to reproduce it. Basically we release objects because the window goes away or the worker is released, but in the meantime we have a runnable in the queue for finishing the body consumption. When that runnable is executed, we crash because data has been already released.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c5cc1b39c89
Avoid FetchConsumer::ContinueConsumeBody() to be called more than once, r=bkelly

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/7c5cc1b39c89

Comment 7

[Intermittent Failures Robot]

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

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

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

Repository breakdown:
* autoland: 9
* mozilla-inbound: 8
* mozilla-central: 6
* mozilla-beta: 5
* mozilla-esr52: 3
* try: 1

Platform breakdown:
* linux32: 11
* windows7-32-vm: 9
* linux64: 6
* linux64-stylo: 3
* windows8-64: 2
* linux32-nightly: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1375749&startday=2017-06-26&endday=2017-07-02&tree=all

Comment 9

[Intermittent Failures Robot]

20 failures in 656 pushes (0.03 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* mozilla-inbound: 6
* autoland: 5
* try: 3
* mozilla-esr52: 3
* mozilla-central: 2
* mozilla-beta: 1

Platform breakdown:
* linux64-stylo: 11
* linux32: 4
* windows7-32-vm: 3
* windows8-64: 1
* linux64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1375749&startday=2017-07-03&endday=2017-07-09&tree=all

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

Do you want to request uplift to beta 55?

Comment 11

[Liz Henry (:lizzard) (relman/hg->git project)]

Looks like this is taken care of in bug 1371889.

Comment 12

[Intermittent Failures Robot]

9 failures in 720 pushes (0.013 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* try: 4
* mozilla-inbound: 2
* autoland: 2
* mozilla-esr52: 1

Platform breakdown:
* linux64: 4
* linux64-stylo: 3
* linux32: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1375749&startday=2017-07-10&endday=2017-07-16&tree=all

Comment 13

[Ryan VanderMeulen [:RyanVM]]

This won't be landing on 55.