Closed Bug 1265610 Opened 8 years ago Closed 8 years ago

test_postMessages.html and some dom/filesystem tests are going to permafail when Gecko 48 merges to Aurora

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox46 --- unaffected
firefox47 + fixed
firefox48 + fixed

People

(Reporter: RyanVM, Assigned: baku)

Details

(Whiteboard: btpp-active)

Attachments

(1 file)

[Tracking Requested - why for this release]: Permafail when Gecko 48 merges to Aurora next week.

Merge day is in a week, so please investigate ASAP.

https://treeherder.mozilla.org/logviewer.html#?job_id=19621451&repo=try

09:45:12     INFO -  1267 INFO TEST-START | dom/base/test/test_postMessages.html
09:45:12     INFO -  JavaScript error: http://mochi.test:8888/tests/dom/base/test/test_postMessages.html, line 109: TypeError: fileList.getFilesAndDirectories is not a function
09:50:37     INFO -  TEST-INFO | started process screentopng
09:50:37     INFO -  TEST-INFO | screentopng: exit 0
09:50:37     INFO -  1268 INFO TEST-PASS | dom/base/test/test_postMessages.html | fileName should be prefs.js
09:50:37     INFO -  1269 INFO TEST-UNEXPECTED-FAIL | dom/base/test/test_postMessages.html | Test timed out.
09:50:37     INFO -      reportError@SimpleTest/TestRunner.js:114:7
09:50:37     INFO -      TestRunner._checkForHangs@SimpleTest/TestRunner.js:135:7
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:156:5
09:50:37     INFO -      TestRunner.runTests@SimpleTest/TestRunner.js:367:5
09:50:37     INFO -      RunSet.runtests@SimpleTest/setup.js:186:3
09:50:37     INFO -      RunSet.runall@SimpleTest/setup.js:165:5
09:50:37     INFO -      hookupTests@SimpleTest/setup.js:258:5
09:50:37     INFO -  parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
09:50:37     INFO -  getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
09:50:37     INFO -  EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
09:50:37     INFO -      hookup@SimpleTest/setup.js:238:5
09:50:37     INFO -  EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&hideResultsTable=1&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp:11:1
09:50:38     INFO -  MEMORY STAT | vsize 843MB | residentFast 197MB | heapAllocated 81MB
09:50:38     INFO -  1270 INFO TEST-OK | dom/base/test/test_postMessages.html | took 325676ms
Flags: needinfo?(amarchesini)
And some other dom/filesystem tests.
https://treeherder.mozilla.org/logviewer.html#?job_id=19623169&repo=try
Summary: test_postMessages.html is going to permafail when Gecko 48 merges to Aurora → test_postMessages.html and some dom/filesystem tests are going to permafail when Gecko 48 merges to Aurora
Assignee: nobody → amarchesini
Flags: needinfo?(amarchesini)
Attached patch test.patchSplinter Review
Attachment #8742707 - Flags: review?(bugs)
Whiteboard: btpp-active
Comment on attachment 8742707 [details] [diff] [review]
test.patch


>+function setup_tests() {
>+  SpecialPowers.pushPrefEnv({"set": [["dom.input.dirpicker", true]]}, next());
>+}
Did you actually test this? As far as I see, pushPrefEnv takes function as a param, and this particular next() doesn't return such.
Make that next.
Attachment #8742707 - Flags: review?(bugs) → review+
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.