Bug 163972
Summary: | REGRESSION(r205345-r205394): [GTK] Layout test fast/canvas/canvas-gradient-addStop-error.html is failing | ||
---|---|---|---|
Product: | WebKit | Reporter: | Michael Catanzaro <mcatanzaro> |
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | bugs-noreply, magomez, Ms2ger |
Priority: | P2 | ||
Version: | Other | ||
Hardware: | PC | ||
OS: | Linux |
Michael Catanzaro
Layout test fast/canvas/canvas-gradient-addStop-error.html has been failing since r205345-r205394 (inclusive). Unfortunately that is a huge range.
--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/fast/canvas/canvas-gradient-addStop-error-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/fast/canvas/canvas-gradient-addStop-error-actual.txt
@@ -7,8 +7,8 @@
PASS gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') threw exception SyntaxError (DOM Exception 12): The string did not match the expected pattern..
PASS gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') threw exception SyntaxError (DOM Exception 12): The string did not match the expected pattern..
Switching to dashboard compatibility mode. Invalid color strings should no longer cause an exception.
-PASS gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') is undefined.
-PASS gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') is undefined.
+FAIL gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') should be undefined. Threw exception SyntaxError (DOM Exception 12): The string did not match the expected pattern.
+FAIL gradient.addColorStop(1, 'rgb(NaN%, NaN%, NaN%)') should be undefined. Threw exception SyntaxError (DOM Exception 12): The string did not match the expected pattern.
PASS successfullyParsed is true
TEST COMPLETE
Updating expectations accordingly.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Ms2ger (he/him; ⌚ UTC+1/+2)
I think we should wontfix this and skip the test; the code it tests for only exists if ENABLE(DASHBOARD_SUPPORT) is true.
Miguel Gomez
This is passing now. Updating expectations and closing bug.