1525785 - windows/aarch64 - toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P5)

Description

[Treeherder Bug Filer]

#[markdown(off)]
Filed by: egao [at] mozilla.com

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

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

Issue observed during win10/aarch64 run.

17:47:36 INFO - <<<<<<<
17:47:36 INFO - TEST-START | toolkit/components/extensions/test/xpcshell/test_ext_native_messaging.js
17:47:38 INFO - TEST-PASS | toolkit/components/extensions/test/xpcshell/test_ext_native_messaging.js | took 3861ms
17:47:38 INFO - TEST-START | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js
17:47:39 WARNING - TEST-UNEXPECTED-FAIL | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | xpcshell return code: 0
17:47:39 INFO - TEST-INFO took 929ms
17:47:39 INFO - >>>>>>>
17:47:39 INFO - PID 18864 | Unable to load \\untrusted-startup-test-dll.dll; LoadLibraryW failed: 126
17:47:39 INFO - (xpcshell/head.js) | test MAIN run_test pending (1)
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 0 pending (2)
17:47:39 INFO - (xpcshell/head.js) | test MAIN run_test finished (2)
17:47:39 INFO - running event loop
17:47:39 INFO - xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | Starting check_remote
17:47:39 INFO - (xpcshell/head.js) | test check_remote pending (2)
17:47:39 INFO - TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | check_remote - [check_remote : 1435] useRemoteWebExtensions matches - true == true
17:47:39 INFO - TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | check_remote - [check_remote : 1435] testing from extension process - false == false
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 0 finished (2)
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 1 pending (2)
17:47:39 INFO - (xpcshell/head.js) | test check_remote finished (2)
17:47:39 INFO - xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | Starting
17:47:39 INFO - (xpcshell/head.js) | test pending (2)
17:47:39 INFO - "Extension attached"
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 1 finished (2)
17:47:39 INFO - PID 18864 | JavaScript strict warning: resource://gre/modules/RemoteWebProgress.jsm, line 246: ReferenceError: reference to undefined property "matchedList"
17:47:39 INFO - "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "ReferenceError: reference to undefined property "matchedList"" {file: "resource://gre/modules/RemoteWebProgress.jsm" line: 246}]"
17:47:39 INFO - TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | - OS is valid - true == true
17:47:39 WARNING - TEST-UNEXPECTED-FAIL | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | - Architecture is valid - false == true
17:47:39 INFO - resource://testing-common/ExtensionXPCShellUtils.jsm:handleResult:321
17:47:39 INFO - resource://gre/modules/ExtensionCommon.jsm:emit:310
17:47:39 INFO - resource://gre/modules/Extension.jsm:receiveMessage:1543
17:47:39 INFO - C:\tasks\task_1549388078\build\tests\xpcshell\head.js:_do_main:224
17:47:39 INFO - C:\tasks\task_1549388078\build\tests\xpcshell\head.js:_execute_test:527
17:47:39 INFO - -e:null:1
17:47:39 INFO - exiting test
17:47:39 INFO - "CONSOLE_MESSAGE: (error) [JavaScript Error: "[Exception... "Abort" nsresult: "0x80004004 (NS_ERROR_ABORT)" location: "JS frame :: C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js :: _abort_failed_test :: line 740" data: no]"]
17:47:39 INFO - _abort_failed_test@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:740:9
17:47:39 INFO - do_report_result@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:847:5
17:47:39 INFO - Assert<@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:54:5
17:47:39 INFO - proto.report@resource://testing-common/Assert.jsm:213:5
17:47:39 INFO - proto.ok@resource://testing-common/Assert.jsm:233:5
17:47:39 INFO - handleResult@resource://testing-common/ExtensionXPCShellUtils.jsm:321:9
17:47:39 INFO - emit@resource://gre/modules/ExtensionCommon.jsm:310:24
17:47:39 INFO - receiveMessage@resource://gre/modules/Extension.jsm:1543:7
17:47:39 INFO - _do_main@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:224:3
17:47:39 INFO - _execute_test@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:527:5
17:47:39 INFO - @-e:1:1
17:47:39 INFO - "
17:47:39 WARNING - TEST-UNEXPECTED-FAIL | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | - Extension left running at test shutdown - "running" == "unloaded"
17:47:39 INFO - resource://testing-common/ExtensionXPCShellUtils.jsm:ExtensionWrapper/<:250
17:47:39 INFO - C:\tasks\task_1549388078\build\tests\xpcshell\head.js:_execute_test/<:591
17:47:39 INFO - exiting test
17:47:39 INFO - NS_ERROR_ABORT:
17:47:39 INFO - _abort_failed_test@C:\tasks\task_1549388078\build\tests\xpcshell\head.js:740:9
17:47:39 INFO - do_report_result@C:\tasks\task_1549388078\build\tests\xpcshell\head.js:847:5
17:47:39 INFO - Assert<@C:\tasks\task_1549388078\build\tests\xpcshell\head.js:54:5
17:47:39 INFO - proto.report@resource://testing-common/Assert.jsm:213:5
17:47:39 INFO - equal@resource://testing-common/Assert.jsm:249:3
17:47:39 INFO - ExtensionWrapper/<@resource://testing-common/ExtensionXPCShellUtils.jsm:250:9
17:47:39 INFO - _execute_test/<@C:\tasks\task_1549388078\build\tests\xpcshell\head.js:591:28
17:47:39 INFO - async*_execute_test@C:\tasks\task_1549388078\build\tests\xpcshell\head.js:588:4
17:47:39 INFO - @-e:1:1
17:47:39 INFO - TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | - runtime.getPlatformInfo - true == true
17:47:39 INFO - TEST-PASS | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js | - test result correct - "runtime.getPlatformInfo" == "runtime.getPlatformInfo"
17:47:39 INFO - "CONSOLE_MESSAGE: (error) [JavaScript Error: "[Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFile.remove]" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: resource://testing-common/ExtensionXPCShellUtils.jsm :: init/< :: line 749" data: no]"]
17:47:39 INFO - init/<@resource://testing-common/ExtensionXPCShellUtils.jsm:749:9
17:47:39 INFO - _execute_test/<@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:591:28
17:47:39 INFO - async*_execute_test@C:\\tasks\\task_1549388078\\build\\tests\\xpcshell\\head.js:588:4
17:47:39 INFO - @-e:1:1
17:47:39 INFO - "
17:47:39 INFO - PID 18864 | [Parent 18864, Gecko_IOThread] WARNING: pipe
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 2 pending (2)
17:47:39 INFO - PID 18864 | error: 109: file z:/build/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 332
17:47:39 INFO - (xpcshell/head.js) | test finished (2)
17:47:39 INFO - (xpcshell/head.js) | test run_next_test 2 finished (1)
17:47:39 INFO - exiting test
17:47:39 INFO - PID 18864 | [Child 15080, Chrome_ChildThread] WARNING: pipe error: 109: file z:/build/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 332
17:47:39 INFO - PID 18864 | [Child 15080, Chrome_ChildThread] WARNING: p[GPU 26168, Chrome_Chi
17:47:39 INFO - PID 18864 | ###!!! [Child][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
17:47:39 INFO - PID 18864 | ldThread] WARNING: pipe error: 109: file z:/bu
17:47:39 INFO - PID 18864 | ###!!! [Child][MessageChannel::SendAndWait] Error: Channel error: cannot send/recv
17:47:39 INFO - <<<<<<<

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

this looks to be a simple test case update that is needed:
https://searchfox.org/mozilla-central/source/toolkit/components/extensions/test/xpcshell/test_ext_runtime_getPlatformInfo.js#8

let validArchs = ["arm", "x86-32", "x86-64"];

I am not sure what arch we have, but :egao, can you figure it out and update this test case?

Comment 2

[Intermittent Failures Robot]

1 failures in 2658 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 1

Platform breakdown:

• windows10-aarch64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1525785&startday=2019-02-04&endday=2019-02-10&tree=all

Comment 3

[Edwin Takahashi (:egao | infrequent contributor)]

Similar (possible) solution mentioned by :m_kato in Bug 1525416. Added the necessary architecture and test jobs is pending for xpcshell-6.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a397ef4e73ead67f0ab8f1c7ae098d1d079d0ce

Hopefully this does the trick for the platform errors.

Comment 4

[Edwin Takahashi (:egao | infrequent contributor)]

Changes to let validArchs = ["arm", "x86-32", "x86-64"]; to include aarch64 appears to have resolved most of the errors that appears opt-xpcshell-6.

The only error that remains is TEST-UNEXPECTED-FAIL | toolkit/components/ctypes/tests/unit/test_jsctypes.js | xpcshell return code: -1073741819, but the exit code is shared with https://bugzilla.mozilla.org/show_bug.cgi?id=1525416 and https://bugzilla.mozilla.org/show_bug.cgi?id=1526022.

:jmaher how should we proceed with this bug? Close this and wait for the main bug at 1525416 with :m_kato to approve changes to validArchs, or would you prefer I just push that change to m-c?

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

we should create a patch for editing validArchs and get that landed, jsctypes is handled elsewhere!

Comment 6

[Edwin Takahashi (:egao | infrequent contributor)]

Attachment: Bug 1525785 - include aarch64 in list of validArchs for xpcshell test that checks architecture r=jmaher,gbrown

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/af0c5b962fca
include aarch64 in list of validArchs for xpcshell test that checks architecture r=jmaher,gbrown

Comment 8

[Razvan Maries]

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

Comment 9

[Vlad Jiman]

Will manual validation from the QA team be required for this fix?

Comment 10

[Edwin Takahashi (:egao | infrequent contributor)]

:vlad - I do not anticipate that is the case, as this change should only apply to the CI environment. Pinging :jmaher to see if he has a differing opinion.

Comment 11

[Joel Maher ( :jmaher ) (UTC -8)]

agreed, this is a test only change, not a product change.

Comment 12

[Vlad Jiman]

Thank you.