Closed Bug 1388490 Opened 7 years ago Closed 7 years ago

Intermittent docshell/test/navigation/test_bug13871.html,test_openWindow.html,test_not-opener.html | java-exception java.lang.NullPointerException at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016)

Tracking

(firefox55 unaffected, firefox56 unaffected, firefox57 fixed)

Status:

RESOLVED FIXED

Milestone:

Firefox 57

Tracking Flags:

Tracking

Status

firefox55

---

unaffected

firefox56

---

unaffected

firefox57

---

fixed

People

(Reporter: intermittent-bug-filer, Assigned: jwu)

References

Details

(Keywords: crash, intermittent-failure, Whiteboard: [stockwell fixed:product][FNC][SPT57.2][MVP])

Attachments

(1 file)

Bug 1388490 - Cancel animation set if necessary before re-start it. 7 years ago Jing-wei Wu [:jwu] 59 bytes, text/x-review-board-request	walkingice : review+	Details

Treeherder Bug Filer

Reporter

Description

•

7 years ago

treeherder

Filed by: wkocher [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=121650501&repo=autoland

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

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Updated

•

7 years ago

Summary: Intermittent docshell/test/navigation/test_bug13871.html | java-exception java.lang.NullPointerException at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016) → Intermittent docshell/test/navigation/test_bug13871.html,test_openWindow.html | java-exception java.lang.NullPointerException at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016)

Phil Ringnalda (:philor)

Updated

•

7 years ago

Component: Security → General

Product: Core → Firefox for Android

Version: unspecified → Trunk

Comment hidden (Intermittent Failures Robot)

15 failures in 166 pushes (0.09 failures/push) were associated with this bug yesterday.   

Repository breakdown:
* autoland: 8
* mozilla-inbound: 4
* mozilla-central: 3

Platform breakdown:
* android-4-3-armv7-api15: 15

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1388490&startday=2017-08-11&endday=2017-08-11&tree=all

Comment hidden (Intermittent Failures Robot)

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

This is the #35 most frequent failure this week.  

** 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: 22
* mozilla-inbound: 14
* try: 4
* mozilla-central: 3

Platform breakdown:
* android-4-3-armv7-api15: 43

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

Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)

Updated

•

7 years ago

Summary: Intermittent docshell/test/navigation/test_bug13871.html,test_openWindow.html | java-exception java.lang.NullPointerException at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016) → Intermittent docshell/test/navigation/test_bug13871.html,test_openWindow.html,test_not-opener.html | java-exception java.lang.NullPointerException at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016)

Geoff Brown [:gbrown] (pto Apr 11-18)

Comment 3

•

7 years ago

This seems typical:

08-08 05:07:17.562   792   792 E GeckoCrashHandler: >>> REPORTING UNCAUGHT EXCEPTION FROM THREAD 1 ("main")
08-08 05:07:17.562   792   792 E GeckoCrashHandler: java.lang.NullPointerException
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1016)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.animation.ValueAnimator.access$400(ValueAnimator.java:50)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.animation.ValueAnimator$AnimationHandler.doAnimationFrame(ValueAnimator.java:630)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.animation.ValueAnimator$AnimationHandler.run(ValueAnimator.java:646)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:749)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.view.Choreographer.doCallbacks(Choreographer.java:562)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.view.Choreographer.doFrame(Choreographer.java:531)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:735)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.os.Handler.handleCallback(Handler.java:730)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.os.Handler.dispatchMessage(Handler.java:92)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.os.Looper.loop(Looper.java:137)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at android.app.ActivityThread.main(ActivityThread.java:5103)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at java.lang.reflect.Method.invokeNative(Native Method)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at java.lang.reflect.Method.invoke(Method.java:525)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
08-08 05:07:17.562   792   792 E GeckoCrashHandler: 	at dalvik.system.NativeStart.main(Native Method)

Geoff Brown [:gbrown] (pto Apr 11-18)

Comment 4

•

7 years ago

https://treeherder.mozilla.org/#/jobs?repo=autoland&filter-searchStr=android%20opt-mochitest-1&tochange=6df7dff00b94cf7232b7d507eee373fe6b0435f0&fromchange=8d02d9abe10eb6b5ddcc4ac695bb0f966b7babac

suggests that this frequent intermittent failure started with the changes for bug 1386975.

:jwu - Please investigate.

Blocks: 1386975

Flags: needinfo?(topwu.tw)

Whiteboard: [stockwell needswork]

Wesly Huang (EPM)

Updated

•

7 years ago

Whiteboard: [stockwell needswork] → [stockwell needswork][FNC][SPT57.2][MVP]

Comment hidden (Intermittent Failures Robot)

15 failures in 188 pushes (0.08 failures/push) were associated with this bug yesterday.   

Repository breakdown:
* mozilla-inbound: 5
* autoland: 5
* mozilla-central: 3
* try: 2

Platform breakdown:
* android-4-3-armv7-api15: 15

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

Comment hidden (Intermittent Failures Robot)

74 failures in 949 pushes (0.078 failures/push) were associated with this bug in the last 7 days. 

This is the #19 most frequent failure this week.  

** 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: 30
* mozilla-inbound: 24
* mozilla-central: 14
* try: 6

Platform breakdown:
* android-4-3-armv7-api15: 74

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1388490&startday=2017-08-14&endday=2017-08-20&tree=all

Joel Maher ( :jmaher ) (UTC -8)

Comment 7

•

7 years ago

:jwu, any updates here, it has been 8 days since the needinfo, not sure if you were on holiday.  With this frequency I am eager to backout your original patch, please comment in here with an estimated time to resolve this bug.

Jing-wei Wu [:jwu]

Assignee

Comment 8

•

7 years ago

Hi Joel,

Looks like this crash is caused by our new visual refresh, a.k.a Photon. Since not all UI changes are landed(bug 1391177, bug 1390735, bug 1389164, etc), we would like to go back to check this issue after we finish all major UI changes in this week.

In the meanwhile, SoftVision have manually helped us test Photon on Nightly for 2 weeks, with different devices and different Android version, so far we didn't receive any crash report like this from them. 

I guess this might be just a false positive alarm, which means we have to modify the test case to fit the new visual refresh.

Flags: needinfo?(topwu.tw)

Joel Maher ( :jmaher ) (UTC -8)

Comment 9

•

7 years ago

:jwu, are we expecting all the photon UI changes to be landed this week and starting next week we can look at the failure rates a nd adjust the test case if needed?

Flags: needinfo?(topwu.tw)

Jing-wei Wu [:jwu]

Assignee

Comment 10

•

7 years ago

Hi Joel,

Yes, most of the major UI changes are landed(a few of them are still under reviewing), we should have more time next week for bugs fix and test cases adjustment.

Assignee: nobody → topwu.tw

tracking-fennec: --- → ?

Flags: needinfo?(topwu.tw)

Comment hidden (Intermittent Failures Robot)

28 failures in 196 pushes (0.143 failures/push) were associated with this bug yesterday.   

Repository breakdown:
* autoland: 12
* mozilla-inbound: 9
* try: 4
* mozilla-central: 3

Platform breakdown:
* android-4-3-armv7-api15: 28

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

Comment hidden (Intermittent Failures Robot)

95 failures in 908 pushes (0.105 failures/push) were associated with this bug in the last 7 days. 

This is the #15 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: 47
* mozilla-inbound: 25
* try: 13
* mozilla-central: 10

Platform breakdown:
* android-4-3-armv7-api15: 95

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1388490&startday=2017-08-21&endday=2017-08-27&tree=all

Comment hidden (mozreview-request)

Review commit: https://reviewboard.mozilla.org/r/173134/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/173134/

Jing-wei Wu [:jwu]

Assignee

Comment 14

•

7 years ago

Try result with this patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f6d5862abadf07edb32c87263be57873b1189f6

Julian Chu [:walkingice]

Comment 15

•

7 years ago

mozreview-review

Comment on attachment 8901700 [details]
Bug 1388490 - Cancel animation set if necessary before re-start it.

https://reviewboard.mozilla.org/r/173134/#review178468

Attachment #8901700 - Flags: review?(walkingice0204) → review+

Pulsebot

Comment 16

•

7 years ago

Pushed by topwu.tw@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/df7f29ed1a89
Cancel animation set if necessary before re-start it. r=walkingice

Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)

Comment 17

•

7 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/df7f29ed1a89

Status: NEW → RESOLVED

Closed: 7 years ago

status-firefox57: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → Firefox 57

Ryan VanderMeulen [:RyanVM]

Updated

•

7 years ago

status-firefox55: --- → unaffected

status-firefox56: --- → unaffected

Keywords: crash

Joel Maher ( :jmaher ) (UTC -8)

Updated

•

7 years ago

Whiteboard: [stockwell needswork][FNC][SPT57.2][MVP] → [stockwell fixed:product][FNC][SPT57.2][MVP]

Comment hidden (Intermittent Failures Robot)

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

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

Platform breakdown:
* android-4-3-armv7-api15: 11

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

James Willcox (:snorp) (jwillcox@mozilla.com) (he/him)

Updated

•

7 years ago

tracking-fennec: ? → ---

Nobody; OK to take it and work on it

Updated

•

3 years ago

Product: Firefox for Android → Firefox for Android Graveyard

You need to log in before you can comment on or make changes to this bug.