Title: [117169] trunk/LayoutTests
- Revision
- 117169
- Author
- [email protected]
- Date
- 2012-05-15 15:33:19 -0700 (Tue, 15 May 2012)
Log Message
Result of fast/dom/DOMException/stack-trace.html depends on path (test introduced in r117016)
https://bugs.webkit.org/show_bug.cgi?id=86442
Reviewed by Darin Adler.
This fixes the test to not print the invalid value (which includes file paths) for the JSC fail case.
* fast/dom/DOMException/stack-trace-expected.txt:
* fast/dom/DOMException/stack-trace.html:
* platform/chromium/fast/dom/DOMException/stack-trace-expected.txt:
* platform/gtk/test_expectations.txt:
* platform/qt/Skipped:
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (117168 => 117169)
--- trunk/LayoutTests/ChangeLog 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/ChangeLog 2012-05-15 22:33:19 UTC (rev 117169)
@@ -1,3 +1,18 @@
+2012-05-15 Erik Arvidsson <[email protected]>
+
+ Result of fast/dom/DOMException/stack-trace.html depends on path (test introduced in r117016)
+ https://bugs.webkit.org/show_bug.cgi?id=86442
+
+ Reviewed by Darin Adler.
+
+ This fixes the test to not print the invalid value (which includes file paths) for the JSC fail case.
+
+ * fast/dom/DOMException/stack-trace-expected.txt:
+ * fast/dom/DOMException/stack-trace.html:
+ * platform/chromium/fast/dom/DOMException/stack-trace-expected.txt:
+ * platform/gtk/test_expectations.txt:
+ * platform/qt/Skipped:
+
2012-05-15 Brady Eidson <[email protected]>
https://bugs.webkit.org/show_bug.cgi?id=86527
Modified: trunk/LayoutTests/fast/dom/DOMException/stack-trace-expected.txt (117168 => 117169)
--- trunk/LayoutTests/fast/dom/DOMException/stack-trace-expected.txt 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/fast/dom/DOMException/stack-trace-expected.txt 2012-05-15 22:33:19 UTC (rev 117169)
@@ -6,10 +6,7 @@
PASS e.name is "HIERARCHY_REQUEST_ERR"
PASS e.message is "HIERARCHY_REQUEST_ERR: DOM Exception 3"
PASS e.code is 3
-FAIL e.stack should be 42 (of type number). Was appendChild@[native code]
-innerFunction@file:///src/WebKit/LayoutTests/fast/dom/DOMException/stack-trace.html:17
-outerFunction@file:///src/WebKit/LayoutTests/fast/dom/DOMException/stack-trace.html:21
-global code@file:///src/WebKit/LayoutTests/fast/dom/DOMException/stack-trace.html:27 (of type string).
+FAIL e.stack === 42 should be true. Was false.
PASS successfullyParsed is true
TEST COMPLETE
Modified: trunk/LayoutTests/fast/dom/DOMException/stack-trace.html (117168 => 117169)
--- trunk/LayoutTests/fast/dom/DOMException/stack-trace.html 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/fast/dom/DOMException/stack-trace.html 2012-05-15 22:33:19 UTC (rev 117169)
@@ -40,7 +40,9 @@
shouldBe('e.code', '3');
e.stack = 42;
-shouldBe('e.stack', '42');
+// Don't use shouldBe because the FAIL case still happens on JSC and we don't want the output
+// to depend on the path.
+shouldBeTrue('e.stack === 42');
</script>
<script src=""
Modified: trunk/LayoutTests/platform/chromium/fast/dom/DOMException/stack-trace-expected.txt (117168 => 117169)
--- trunk/LayoutTests/platform/chromium/fast/dom/DOMException/stack-trace-expected.txt 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/platform/chromium/fast/dom/DOMException/stack-trace-expected.txt 2012-05-15 22:33:19 UTC (rev 117169)
@@ -6,7 +6,7 @@
PASS e.name is "HIERARCHY_REQUEST_ERR"
PASS e.message is "HIERARCHY_REQUEST_ERR: DOM Exception 3"
PASS e.code is 3
-PASS e.stack is 42
+PASS e.stack === 42 is true
PASS successfullyParsed is true
TEST COMPLETE
Modified: trunk/LayoutTests/platform/gtk/test_expectations.txt (117168 => 117169)
--- trunk/LayoutTests/platform/gtk/test_expectations.txt 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/platform/gtk/test_expectations.txt 2012-05-15 22:33:19 UTC (rev 117169)
@@ -1318,8 +1318,6 @@
BUGWK86328 : media/video-controls-toggling.html = FAIL
BUGWK86328 : media/video-controls-rendering-toggle-display-none.html = FAIL
-BUGWK86442 : fast/dom/DOMException/stack-trace.html = FAIL
-
BUGWK86443 : plugins/npruntime/delete-plugin-within-hasProperty-return-false.html = FAIL
BUGWK86443 : plugins/npruntime/delete-plugin-within-hasProperty-return-true.html = FAIL
BUGWK86443 : plugins/npruntime/delete-plugin-within-setProperty.html = FAIL
Modified: trunk/LayoutTests/platform/qt/Skipped (117168 => 117169)
--- trunk/LayoutTests/platform/qt/Skipped 2012-05-15 22:05:53 UTC (rev 117168)
+++ trunk/LayoutTests/platform/qt/Skipped 2012-05-15 22:33:19 UTC (rev 117169)
@@ -2551,10 +2551,6 @@
# Inspector only supports evaluation in content script world with v8, see https://bugs.webkit.org/show_bug.cgi?id=85709
inspector/extensions/extensions-eval-content-script.html
-# Result of fast/dom/DOMException/stack-trace.html depends on path (test introduced in r117016)
-# https://bugs.webkit.org/show_bug.cgi?id=86442
-fast/dom/DOMException/stack-trace.html
-
# [Qt][GTK] New fast/multicol/split-in-top-margin.html fails
# https://bugs.webkit.org/show_bug.cgi?id=86445
fast/multicol/split-in-top-margin.html
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes