1447259 - Intermittent devtools/client/inspector/animation/test/browser_animation_summary-graph_computed-timing-path_different-timescale.js | A promise chain failed to handle a rejection: Minified React error #188

Status: RESOLVED FIXED

(DevTools :: Inspector: Animations, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: shindli [at] mozilla.com

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

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

Comment 1

[Intermittent Failures Robot]

40 failures in 822 pushes (0.049 failures/push) were associated with this bug in the last 7 days. 

This is the #41 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: 17
* mozilla-inbound: 15
* mozilla-central: 5
* try: 3

Platform breakdown:
* osx-10-10: 32
* windows10-64: 5
* windows7-32: 2
* linux64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1447259&startday=2018-03-20&endday=2018-03-26&tree=trunk

Comment 2

[Noemi Erli[:noemi_erli]]

There are 33 failures in the past 7 days. 
Platforms:
- most of them are happening on OS X 10.10 debug, but we also have some occurrences on Windows 7 and windows10-64, both debug.
Recent log failure: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=170682180&lineNumber=9818
Relevant part of the log: 
23:04:22     INFO -  Buffered messages finished
23:04:22    ERROR -  310 INFO TEST-UNEXPECTED-FAIL | devtools/client/inspector/animation/test/browser_animation_summary-graph_computed-timing-path_different-timescale.js | A promise chain failed to handle a rejection: Minified React error #188; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=188 for the full message or use the non-minified dev environment for full errors and additional helpful warnings. - stack: reactProdInvariant@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:32:15
23:04:22     INFO -  findCurrentFiberUsingSlowPath@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:3003:7
23:04:22     INFO -  findCurrentHostFiber@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:3072:23
23:04:22     INFO -  findHostInstance@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10633:21
23:04:22     INFO -  findDOMNode@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:12847:14
23:04:22     INFO -  updateState@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/components/graph/SummaryGraphPath.js:174:20
23:04:22     INFO -  async*componentWillReceiveProps@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/components/graph/SummaryGraphPath.js:53:5
23:04:22     INFO -  callComponentWillReceiveProps@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:6244:5
23:04:22     INFO -  updateClassInstance@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:6419:7
23:04:22     INFO -  updateClassComponent@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:7537:22
23:04:22     INFO -  beginWork@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:7883:16
23:04:22     INFO -  performUnitOfWork@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:9706:16
23:04:22     INFO -  workLoop@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:9753:26
23:04:22     INFO -  renderRoot@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:9832:9
23:04:22     INFO -  performWorkOnRoot@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10465:24
23:04:22     INFO -  performWork@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10418:7
23:04:22     INFO -  requestWork@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10329:7
23:04:22     INFO -  scheduleWorkImpl@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10188:11
23:04:22     INFO -  scheduleWork@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:10152:12
23:04:22     INFO -  enqueueSetState@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-dom.js:6131:7
23:04:22     INFO -  Component.prototype.setState@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react.js:346:3
23:04:22     INFO -  onStateChange@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/react-redux.js:1412:11
23:04:22     INFO -  dispatch@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/vendor/redux.js:416:7
23:04:22     INFO -  log/</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/log.js:17:5
23:04:22     INFO -  exports.history/</</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/history.js:21:5
23:04:22     INFO -  waitUntilService/</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/wait-service.js:59:18
23:04:22     INFO -  promiseMiddleware/</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/promise.js:15:14
23:04:22     INFO -  thunk/</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/thunk.js:16:9
23:04:22     INFO -  task/</<@resource://devtools/shared/base-loader.js -> resource://devtools/client/shared/redux/middleware/task.js:36:12
23:04:22     INFO -  updateState@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:574:5
23:04:22     INFO -  setAnimationsCurrentTime@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:376:7
23:04:22     INFO -  Async*onCurrentTimeTimerUpdated@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:278:7
23:04:22     INFO -  next@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:58:7
23:04:22     INFO -  FrameRequestCallback*next@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:63:5
23:04:22     INFO -  FrameRequestCallback*next@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:63:5
23:04:22     INFO -  FrameRequestCallback*next@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:63:5
23:04:22     INFO -  FrameRequestCallback*next@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:63:5
23:04:22     INFO -  start@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/current-time-timer.js:67:5
23:04:22     INFO -  startAnimationsCurrentTimeTimer@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:522:5
23:04:22     INFO -  updateState@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:577:7
23:04:22     INFO -  update@resource://devtools/shared/base-loader.js -> resource://devtools/client/inspector/animation/animation.js:541:7
23:04:22     INFO -  Async*emit@resource://devtools/shared/base-loader.js -> resource://devtools/shared/event-emitter.js:178:37
23:04:22     INFO -  emit@resource://devtools/shared/base-loader.js -> resource://devtools/shared/event-emitter.js:255:29
23:04:22     INFO -  setNodeFront@resource://devtools/shared/base-loader.js -> resource://devtools/client/framework/selection.js:128:5
23:04:22     INFO -  selectNode@chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js:208:3
23:04:22     INFO -  async*selectNodeAndWaitForAnimations@chrome://mochitests/content/browser/devtools/client/inspector/animation/test/head.js:327:9
23:04:22     INFO -  async*@chrome://mochitests/content/browser/devtools/client/inspector/animation/test/browser_animation_summary-graph_computed-timing-path_different-timescale.js:13:9
23:04:22     INFO -  Async*Tester_execTest/<@chrome://mochikit/content/browser-test.js:1076:21
23:04:22     INFO -  TaskImpl_run@resource://gre/modules/Task.jsm:326:42
23:04:22     INFO -  TaskImpl@resource://gre/modules/Task.jsm:275:3
23:04:22     INFO -  asyncFunction@resource://gre/modules/Task.jsm:247:14
23:04:22     INFO -  Task_spawn@resource://gre/modules/Task.jsm:161:12
23:04:22     INFO -  Tester_execTest@chrome://mochikit/content/browser-test.js:1067:9
23:04:22     INFO -  Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:967:9
23:04:22     INFO -  SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:795:59
23:04:22     INFO -  Rejection date: Tue Mar 27 2018 23:04:21 GMT+0000 (Coordinated Universal Time) - false == true - JS frame :: resource://testing-common/PromiseTestUtils.jsm :: assertNoUncaughtRejections :: line 257
23:04:22     INFO -  Stack trace:
23:04:22     INFO -      resource://testing-common/PromiseTestUtils.jsm:assertNoUncaughtRejections:257
23:04:22     INFO -      chrome://mochikit/content/browser-test.js:Tester_execTest/<:1098
23:04:22     INFO -      Tester_execTest@chrome://mochikit/content/browser-test.js:1067:9
23:04:22     INFO -      Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:967:9
23:04:22     INFO -      SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:795:59
:daisuke Can you please take a look here?

Comment 3

[Daisuke Akatsuka (:daisuke)]

Yes, thanks.

Comment 4

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1447259: Change testing animation since the animation sometimes already finished during waiting for selectNodeAndWaitForAnimations.

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

Comment 5

[Intermittent Failures Robot]

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

This is the #41 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: 18
* mozilla-inbound: 13
* mozilla-central: 6
* try: 2

Platform breakdown:
* osx-10-10: 17
* windows7-32: 15
* windows10-64: 7

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1447259&startday=2018-03-26&endday=2018-04-01&tree=trunk

Comment 6

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 8963121 [details]
Bug 1447259: Change testing animation since the animation sometimes already finished during waiting for selectNodeAndWaitForAnimations.

https://reviewboard.mozilla.org/r/231996/#review238512

::: commit-message-65d4a:1
(Diff revision 1)
> +Bug 1447259: Change testing animation since the animation sometimes already finished during waiting for selectNodeAndWaitForAnimations. r?gl

Change the tested animation in browser_animation_summary-graph_computed-timing-path_different-timescale.js test since the animation can intermittently already be finished during selectNodeAndWaitForAnimations.

Comment 7

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1447259: Change the tested animation in browser_animation_summary-graph_computed-timing-path_different-timescale.js test since the animation can intermittently already be finished during selectNodeAndWaitForAnimations.

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

Comment 8

[Daisuke Akatsuka (:daisuke)]

I will make it to land to the inbound because r+ was cleared after updating a patch.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/77e268b40ed6
Change the tested animation in browser_animation_summary-graph_computed-timing-path_different-timescale.js test since the animation can intermittently already be finished during selectNodeAndWaitForAnimations. r=gl

Comment 10

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 8964452 [details]
Bug 1447259: Change the tested animation in browser_animation_summary-graph_computed-timing-path_different-timescale.js test since the animation can intermittently already be finished during selectNodeAndWaitForAnimations.

https://reviewboard.mozilla.org/r/233180/#review238662

Comment 11

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/77e268b40ed6

Comment 12

[Intermittent Failures Robot]

13 failures in 765 pushes (0.017 failures/push) were associated with this bug in the last 7 days.    

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

Platform breakdown:
* osx-10-10: 7
* windows7-32: 4
* windows10-64: 2

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1447259&startday=2018-04-02&endday=2018-04-08&tree=trunk