Bug 141848

Summary:

Layout Test js/regress-141098.html is failing on 32-bit Machines

Product:

WebKit

Reporter:

Brent Fulgham <bfulgham>

Component:

JavaScriptCore

Assignee:

Michael Saboff <msaboff>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, fpizlo, msaboff, ossy

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

141676

Attachments:

Description	Flags
Patch	ggaren: review+

Description Brent Fulgham 2015-02-20 12:30:34 PST

The following layout test is failing on 32-bit platforms.

js/regress-141098.html

Probable cause:

We only seem to get one of the two expected exceptions. Perhaps 32-bit doesn't execute part of the test?

--- /home/buildbot/slave/win-release-tests/build/layout-test-results/js/regress-141098-expected.txt
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/js/regress-141098-actual.txt
@@ -4,7 +4,6 @@
 
 
 PASS Exception: RangeError: Maximum call stack size exceeded.
-PASS Exception: RangeError: Maximum call stack size exceeded.
 PASS successfullyParsed is true
 
 TEST COMPLETE

Comment 1 Brent Fulgham 2015-02-20 12:40:08 PST

** The following JSC stress test failures have been introduced:
	jsc-layout-tests.yaml/js/script-tests/regress-141098.js.layout
	jsc-layout-tests.yaml/js/script-tests/regress-141098.js.layout-dfg-eager-no-cjit
	jsc-layout-tests.yaml/js/script-tests/regress-141098.js.layout-no-cjit
	jsc-layout-tests.yaml/js/script-tests/regress-141098.js.layout-no-llint

Comment 2 Brent Fulgham 2015-02-20 12:42:16 PST

Skipped failing layout test on Windows:
<https://trac.webkit.org/changeset/180446>

Comment 3 Brent Fulgham 2015-02-20 12:55:54 PST

JSC stress tests now skipped:
<https://trac.webkit.org/changeset/180448>

Comment 4 Michael Saboff 2015-02-20 13:53:10 PST

Created attachment 246997 [details]
Patch

Comment 5 Geoffrey Garen 2015-02-20 14:02:22 PST

Comment on attachment 246997 [details]
Patch

r=me

Comment 6 Michael Saboff 2015-02-20 14:21:58 PST

Committed r180453: <http://trac.webkit.org/changeset/180453>