RESOLVED FIXED 158126
Release JSC test test-observegc.js.layout failing 
https://bugs.webkit.org/show_bug.cgi?id=158126
Summary Release JSC test test-observegc.js.layout failing
Ryan Haddad
Reported 2016-05-26 11:35:24 PDT
<https://build.webkit.org/builders/Apple%20El%20Capitan%20Release%20JSC%20%28Tests%29/builds/6205/steps/jscore-test/logs/stdio> ** The following JSC stress test failures have been introduced: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-ftl jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-ftl-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-ftl-no-cjit jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: Ensures that window.internals.observegc works as expected jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: Exception: ReferenceError: Can't find variable: internals jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: global code@test-observegc.js:5:25 jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: PASS successfullyParsed is true jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: TEST COMPLETE jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: jsc-layout-tests.yaml/js/script-tests/test-observegc.js.layout: ERROR: Unexpected exit code: 3
Attachments
Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2016-05-26 11:57:10 PDT
TIL that "LayoutTests/js" is not an acceptable place to put a "Layout Test", as it is "special"
Brady Eidson
Comment 2 2016-05-26 12:05:12 PDT
http://trac.webkit.org/changeset/201425
Note You need to log in before you can comment on or make changes to this bug.