Closed Bug 1083140 Opened 5 years ago Closed 3 years ago

Intermittent devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | A promise chain failed to handle a rejection: - destroy

Categories

(DevTools :: WebIDE, defect, P3)

x86
Linux
defect

Tracking

(firefox52 wontfix, firefox-esr52 fixed, firefox53 fixed, firefox54 fixed, firefox55 fixed)

RESOLVED FIXED
Firefox 55
Tracking Status
firefox52 --- wontfix
firefox-esr52 --- fixed
firefox53 --- fixed
firefox54 --- fixed
firefox55 --- fixed

People

(Reporter: cbook, Assigned: jmaher)

References

()

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed])

Attachments

(1 file)

Ubuntu VM 12.04 b2g-inbound debug test mochitest-devtools-chrome-2

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

22:28:50 INFO - 8072 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/projecteditor/test/browser_projecteditor_contextmenu_01.js | A promise chain failed to handle a rejection: - destroy
Inactive; closing (see bug 1180138).
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
this picked up recently around March 7th, keeping an eye on this bug.
now I have 36 failures in the last week all on linux64-asan e10s:
https://brasstacks.mozilla.com/orangefactor/index.html?display=Bug&bugid=1083140&startday=2017-03-17&endday=2017-03-24&tree=trunk

a recent failure log:
https://brasstacks.mozilla.com/orangefactor/index.html?display=Bug&bugid=1083140&startday=2017-03-17&endday=2017-03-24&tree=trunk

and screenshot:
https://public-artifacts.taskcluster.net/XwxcvWkDQ_ufrVGNfwUmAA/0/public/test_info//mozilla-test-fail-screenshot_ZYa4pN.png

the screenshot yields nothing but a blank screen, here is a clip from the log file:
[task 2017-03-24T09:55:10.454883Z] 09:55:10     INFO - TEST-START | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js
[task 2017-03-24T09:55:10.535460Z] 09:55:10     INFO - TEST-INFO | started process screentopng
[task 2017-03-24T09:55:12.610773Z] 09:55:12     INFO - TEST-INFO | screentopng: exit 0
[task 2017-03-24T09:55:12.614818Z] 09:55:12     INFO - TEST-UNEXPECTED-FAIL | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | A promise chain failed to handle a rejection:  - destroy
[task 2017-03-24T09:55:12.615101Z] 09:55:12     INFO - Stack trace:
[task 2017-03-24T09:55:12.616423Z] 09:55:12     INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: register :: line 199
[task 2017-03-24T09:55:12.617688Z] 09:55:12     INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: completePromise :: line 711
[task 2017-03-24T09:55:12.618813Z] 09:55:12     INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: process :: line 958
[task 2017-03-24T09:55:12.620016Z] 09:55:12     INFO - JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: walkerLoop :: line 806
[task 2017-03-24T09:55:12.622422Z] 09:55:12     INFO - Entering test bound 
[task 2017-03-24T09:55:12.624682Z] 09:55:12     INFO - Building a temporary directory at ProjectEditor1490349310511
[task 2017-03-24T09:55:12.629487Z] 09:55:12     INFO - Adding a project editor tab for editing at: /tmp/ProjectEditor1490349310511
[task 2017-03-24T09:55:12.635117Z] 09:55:12     INFO - Adding a new tab with URL: 'chrome://mochitests/content/browser/devtools/client/projecteditor/test/projecteditor-test.xul'
[task 2017-03-24T09:55:12.637281Z] 09:55:12     INFO - URL 'chrome://mochitests/content/browser/devtools/client/projecteditor/test/projecteditor-test.xul' loading complete
[task 2017-03-24T09:55:12.639709Z] 09:55:12     INFO - must wait for focus
[task 2017-03-24T09:55:12.642349Z] 09:55:12     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | Tab has placeholder iframe for projecteditor - 
[task 2017-03-24T09:55:12.644531Z] 09:55:12     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | ProjectEditor has been initialized - 
[task 2017-03-24T09:55:12.974302Z] 09:55:12     INFO - Console message: [JavaScript Warning: "Property contained reference to invalid variable.  Error in parsing value for ‘background-color’.  Falling back to ‘initial’." {file: "resource://devtools/client/themes/splitters.css" line: 39 column: 1268 source: " var(--devtools-splitter-color)"}]
[task 2017-03-24T09:55:14.752975Z] 09:55:14     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | ProjectEditor has loaded - 
[task 2017-03-24T09:55:14.760784Z] 09:55:14     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has loaded in the projecteditor document - 
[task 2017-03-24T09:55:14.765409Z] 09:55:14     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has loaded in the projecteditor document - 
[task 2017-03-24T09:55:14.770275Z] 09:55:14     INFO - Building a temporary directory at ProjectEditor1490349310511
[task 2017-03-24T09:55:14.794403Z] 09:55:14     INFO - Adding a project editor tab for editing at: /tmp/ProjectEditor1490349310511
[task 2017-03-24T09:55:14.798333Z] 09:55:14     INFO - Adding a new tab with URL: 'chrome://mochitests/content/browser/devtools/client/projecteditor/test/projecteditor-test.xul'
[task 2017-03-24T09:55:15.688476Z] 09:55:15     INFO - URL 'chrome://mochitests/content/browser/devtools/client/projecteditor/test/projecteditor-test.xul' loading complete
[task 2017-03-24T09:55:15.697870Z] 09:55:15     INFO - must wait for focus
[task 2017-03-24T09:55:15.754895Z] 09:55:15     INFO - TEST-PASS | devtools/client/proj
[task 2017-03-24T09:55:15.763131Z] 09:55:15     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | ProjectEditor has been initialized - 
[task 2017-03-24T09:55:16.046773Z] 09:55:16     INFO - Console message: [JavaScript War
[task 2017-03-24T09:55:17.406880Z] 09:55:17     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has NOT loaded in the projecteditor document - 
[task 2017-03-24T09:55:17.415440Z] 09:55:17     INFO - TEST-PASS | devtools/client/proj
ecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has NOT loaded in the projecteditor document - 
[task 2017-03-24T09:55:17.417085Z] 09:55:17     INFO - TEST-PASS | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has loaded in the specified element - 
ecteditor/test/browser_projecteditor_contextmenu_01.js | The menu has loaded in the specified element - 
[task 2017-03-24T09:55:17.441343Z] 09:55:17     INFO - Leaving test bound 
[task 2017-03-24T09:55:18.090759Z] 09:55:18     INFO - GECKO(1640) | MEMORY STAT | vsize 20973802MB | residentFast 592MB
[task 2017-03-24T09:55:18.092614Z] 09:55:18     INFO - TEST-OK | devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | took 7634ms


looking at the test, I don't see an obvious point where we are failing, the messages show us running the test properly, maybe there is an issue with the yield statement or something in the harness?

:jryans, when you are back next week, could you find a little bit of time to look at this?
Flags: needinfo?(jryans)
Summary: Intermittent projecteditor/test/browser_projecteditor_contextmenu_01.js | A promise chain failed to handle a rejection: - destroy → Intermittent devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js | A promise chain failed to handle a rejection: - destroy
Whiteboard: [stockwell needswork]
This test is covering an area we'll probably be removing in the future, so I don't think investigating this is worth it right now.  Could we just disable this test for linux64-asan perhaps?
Flags: needinfo?(jryans)
(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #32)
> This test is covering an area we'll probably be removing in the future, so I
> don't think investigating this is worth it right now.  Could we just disable
> this test for linux64-asan perhaps?

Agree with disabling the test
this patch should do the trick
Attachment #8854554 - Flags: review?(jryans)
Comment on attachment 8854554 [details] [diff] [review]
disable test on asan

Review of attachment 8854554 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, looks good to me! :)
Attachment #8854554 - Flags: review?(jryans) → review+
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d339e21df154
Intermittent devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js. disable on asan. r=jryans
Component: Developer Tools → Developer Tools: WebIDE
https://hg.mozilla.org/mozilla-central/rev/d339e21df154
Status: REOPENED → RESOLVED
Closed: 4 years ago3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
https://hg.mozilla.org/releases/mozilla-aurora/rev/be0e99166797
Flags: in-testsuite-
Whiteboard: [stockwell needswork] → [stockwell fixed]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.