Bug 141848 - Layout Test js/regress-141098.html is failing on 32-bit Machines
Summary: Layout Test js/regress-141098.html is failing on 32-bit Machines
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords:
Depends on:
Blocks: 141676
  Show dependency tree / graph
 
Reported: 2015-02-20 12:30 PST by Brent Fulgham
Modified: 2015-02-24 15:31 PST (History)
4 users (show)

See Also:

Attachments
Patch (4.04 KB, patch)
2015-02-20 13:53 PST, Michael Saboff 		ggaren: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>