summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorager@google.com <ager@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-10 12:32:20 +0000
committerager@google.com <ager@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-10 12:32:20 +0000
commitbc6771854262babec634e005b9c3fa4b82336a81 (patch)
treeb163f6f4e8015d0ca08be9b3c1e9359bccaf941b
parente2cca16d204c1eb184fdca2ca87c9010c47f3c57 (diff)
downloadchromium_src-bc6771854262babec634e005b9c3fa4b82336a81.zip
chromium_src-bc6771854262babec634e005b9c3fa4b82336a81.tar.gz
chromium_src-bc6771854262babec634e005b9c3fa4b82336a81.tar.bz2
Fix test list syntax.
Rebaseline css3/keyframes-rule.html. Review URL: http://codereview.chromium.org/10251 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5103 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--webkit/data/layout_test_results/v8/LayoutTests/css3/README5
-rw-r--r--webkit/data/layout_test_results/v8/LayoutTests/css3/keyframes-rule-expected.txt62
-rw-r--r--webkit/tools/layout_tests/test_lists/mac/tests_fixable.txt104
-rw-r--r--webkit/tools/layout_tests/test_lists/win/tests_fixable.txt98
4 files changed, 164 insertions, 105 deletions
diff --git a/webkit/data/layout_test_results/v8/LayoutTests/css3/README b/webkit/data/layout_test_results/v8/LayoutTests/css3/README
index c75077f..11f7511 100644
--- a/webkit/data/layout_test_results/v8/LayoutTests/css3/README
+++ b/webkit/data/layout_test_results/v8/LayoutTests/css3/README
@@ -2,3 +2,8 @@ transform-value-types
--------------------------
The printing of prototype objects and constructors is implementation dependent.
We print these differently from KJS which is harmless.
+
+keyframes-rule
+--------------------------
+Expects typeof(CSSStyleDeclaration) to be 'object'. In V8 this is a
+constructor function and typeof(CSSStyleDeclaration) is 'function'.
diff --git a/webkit/data/layout_test_results/v8/LayoutTests/css3/keyframes-rule-expected.txt b/webkit/data/layout_test_results/v8/LayoutTests/css3/keyframes-rule-expected.txt
new file mode 100644
index 0000000..64907c8
--- /dev/null
+++ b/webkit/data/layout_test_results/v8/LayoutTests/css3/keyframes-rule-expected.txt
@@ -0,0 +1,62 @@
+This tests the WebKitCSSKeyframeRule and WebKitCSSKeyframesRule interfaces.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS WEBKIT_KEYFRAME_RULE exists on Window object
+PASS WEBKIT_KEYFRAMES_RULE exists on Window object
+
+PASS keyframes1.type is window.CSSRule.WEBKIT_KEYFRAMES_RULE
+PASS keyframes1.name is 'test1'
+
+PASS rules1.length is 2
+PASS rules1.item(0).type is window.CSSRule.WEBKIT_KEYFRAME_RULE
+PASS rules1.item(0).cssText is '0% { left: 10px; }'
+PASS rules1.item(1).cssText is '100% { left: 20px; }'
+
+Insert three new rules
+PASS rules1.length is 5
+PASS rules1.item(0).cssText is '0% { left: 10px; }'
+PASS rules1.item(1).cssText is '100% { left: 20px; }'
+PASS rules1.item(2).cssText is '30% { left: 30px; }'
+PASS rules1.item(3).cssText is '60% { left: 60px; }'
+PASS rules1.item(4).cssText is '20% { left: 50px; }'
+
+PASS keyframes2.type is window.CSSRule.WEBKIT_KEYFRAMES_RULE
+PASS keyframes2.name is 'test2'
+
+PASS rules2.length is 3
+PASS rules2.item(0).type is window.CSSRule.WEBKIT_KEYFRAME_RULE
+PASS rules2.item(0).keyText is '0%'
+PASS rules2.item(1).keyText is '50%'
+PASS rules2.item(2).keyText is '100%'
+PASS rules2.item(0).cssText is '0% { left: 10px; }'
+PASS rules2.item(1).cssText is '50% { left: 30px; }'
+PASS rules2.item(2).cssText is '100% { left: 20px; }'
+
+FAIL typeof(rules2.item(0).style) should be function. Was object.
+PASS rules2.item(0).style.length is 1
+
+Find a rule
+PASS rule.type is window.CSSRule.WEBKIT_KEYFRAME_RULE
+PASS rule.cssText is '50% { left: 30px; }'
+
+Try to find a rule that doesn't exist
+PASS Non-existent rule was not found
+
+Delete a rule
+PASS rules2.length is 2
+PASS rules2.item(0).type is window.CSSRule.WEBKIT_KEYFRAME_RULE
+PASS rules2.item(0).keyText is '0%'
+PASS rules2.item(1).keyText is '100%'
+
+Delete a rule that doesn't exist
+PASS rules2.length is 2
+PASS rules2.item(0).type is window.CSSRule.WEBKIT_KEYFRAME_RULE
+PASS rules2.item(0).keyText is '0%'
+PASS rules2.item(1).keyText is '100%'
+
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
diff --git a/webkit/tools/layout_tests/test_lists/mac/tests_fixable.txt b/webkit/tools/layout_tests/test_lists/mac/tests_fixable.txt
index 6e744ef..4486a7d 100644
--- a/webkit/tools/layout_tests/test_lists/mac/tests_fixable.txt
+++ b/webkit/tools/layout_tests/test_lists/mac/tests_fixable.txt
@@ -79,7 +79,7 @@ V8 # LayoutTests/http/tests/xmlhttprequest/upload-onload-event.html = FAIL
V8 # LayoutTests/http/tests/xmlhttprequest/upload-onprogress-event.html = FAIL
V8 # LayoutTests/http/tests/xmlhttprequest/upload-progress-events.html = FAIL
-# Bug 1341582
+// Bug 1341582
V8 # LayoutTests/svg/dom/animated-tearoff-equality.xhtml = FAIL | PASS
V8 # LayoutTests/svg/dom/animated-tearoff-lifespan.xhtml = FAIL | PASS
@@ -658,37 +658,37 @@ V8 # pending/http/tests/security/cross-frame-access-protocol.html = PASS | TIMEO
V8 # LayoutTests/fast/js/date-DST-time-cusps.html = FAIL | PASS
V8 # LayoutTests/fast/js/date-big-setdate.html = FAIL | PASS
-# These three tests have their results changed on WebKit trunk that matches our
-# current results: http://trac.webkit.org/changeset/36652
-# When we pick up that revision of WebKit, we can remove these exceptions.
-# TODO(pinkerton): these were removed from win, but still fail for Mac
+// These three tests have their results changed on WebKit trunk that matches our
+// current results: http://trac.webkit.org/changeset/36652
+// When we pick up that revision of WebKit, we can remove these exceptions.
+// TODO(pinkerton): these were removed from win, but still fail for Mac
V8 # LayoutTests/fast/events/onunload.html = FAIL
V8 # LayoutTests/fast/events/onunload-window-property.html = FAIL
V8 # LayoutTests/fast/forms/button-state-restore.html = FAIL
-# V8 doesn't stable sort. This is not required and may never be fixed. It is
-# tracked by: http://code.google.com/p/v8/issues/detail?id=90
+// V8 doesn't stable sort. This is not required and may never be fixed. It is
+// tracked by: http://code.google.com/p/v8/issues/detail?id=90
V8 # LayoutTests/fast/js/comparefn-sort-stability.html = FAIL
V8 # LayoutTests/fast/js/sort-stability.html = FAIL
-# Apparently V8 date handling:
-# http://code.google.com/p/v8/issues/detail?id=91
+// Apparently V8 date handling:
+// http://code.google.com/p/v8/issues/detail?id=91
V8 # LayoutTests/fast/js/date-set-to-nan.html = FAIL
-# These require application cache to be enabled. Skip the tests because
-# they're all timing out. TODO(tc): Upstream changes to the test so they
-# fail quickly if window.applicationCache is undefined (then we don't have to
-# skip them).
-# http://crbug.com/2844
+// These require application cache to be enabled. Skip the tests because
+// they're all timing out. TODO(tc): Upstream changes to the test so they
+// fail quickly if window.applicationCache is undefined (then we don't have to
+// skip them).
+// http://crbug.com/2844
V8 # SKIP : LayoutTests/http/tests/appcache = TIMEOUT | FAIL
-# Depends on postMessage in a way we don't support:
-# http://code.google.com/p/chromium/issues/detail?id=2857
+// Depends on postMessage in a way we don't support:
+// http://code.google.com/p/chromium/issues/detail?id=2857
V8 # LayoutTests/http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame-uppercase.html = FAIL
V8 # LayoutTests/http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame.html = FAIL
-# V8 failures as a result of the WebKit merge.
-# http://code.google.com/p/v8/issues/detail?id=92
+// V8 failures as a result of the WebKit merge.
+// http://code.google.com/p/v8/issues/detail?id=92
V8 # LayoutTests/fast/js/constructor-attributes.html = FAIL
V8 # LayoutTests/fast/js/eval-cross-window.html = FAIL
V8 # LayoutTests/fast/js/eval-keyword-vs-function.html = FAIL
@@ -702,12 +702,12 @@ V8 # LayoutTests/fast/js/removing-Cf-characters.html = FAIL
V8 # LayoutTests/fast/js/static-scope-object.html = FAIL
V8 # LayoutTests/fast/js/delete-getters-setters.html = FAIL
-# Console output won't have line numbers until V8 gives us a way to get that
-# information. <http://crbug.com/2960>
+// Console output won't have line numbers until V8 gives us a way to get that
+// information. <http://crbug.com/2960>
V8 # LayoutTests/fast/dom/Window/console-functions.html = FAIL
-# Shadows don't render correctly for these tests.
-# http://code.google.com/p/chromium/issues/detail?id=2969
+// Shadows don't render correctly for these tests.
+// http://code.google.com/p/chromium/issues/detail?id=2969
V8 # LayoutTests/fast/canvas/shadow-offset-1.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-2.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-3.html = FAIL
@@ -717,73 +717,69 @@ V8 # LayoutTests/fast/canvas/shadow-offset-6.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-7.html = FAIL
V8 # LayoutTests/fast/css/shadow-multiple.html = FAIL
-# There appears to be some parsing error. These tests give "successfullyParsed
-# should be true (of type boolean). Was undefined (of type undefined)."
-# http://code.google.com/p/chromium/issues/detail?id=2976
+// There appears to be some parsing error. These tests give "successfullyParsed
+// should be true (of type boolean). Was undefined (of type undefined)."
+// http://code.google.com/p/chromium/issues/detail?id=2976
V8 # LayoutTests/fast/canvas/drawImage-with-negative-source-destination.html = FAIL
-# getImageData isn't supported on canvas.
-# http://code.google.com/p/chromium/issues/detail?id=2974
+// getImageData isn't supported on canvas.
+// http://code.google.com/p/chromium/issues/detail?id=2974
V8 # LayoutTests/fast/canvas/canvas-pattern-behaviour.html = FAIL
V8 # LayoutTests/fast/canvas/canvas-getImageData.html = FAIL
-# We don't implement toDataURL.
-# http://code.google.com/p/chromium/issues/detail?id=2972
+// We don't implement toDataURL.
+// http://code.google.com/p/chromium/issues/detail?id=2972
V8 # LayoutTests/fast/canvas/toDataURL-supportedTypes.html = FAIL
-# layoutTestController.setStopProvisionalFrameLoads needs implementing
-# http://code.google.com/p/chromium/issues/detail?id=2980
+// layoutTestController.setStopProvisionalFrameLoads needs implementing
+// http://code.google.com/p/chromium/issues/detail?id=2980
V8 # LayoutTests/fast/loader/stop-provisional-loads.html = FAIL
-# Our shadows looks wrong. The text not having shadows is to be expected, but
-# it looks like we're missing a red shadow that Safari has.
-# http://code.google.com/p/chromium/issues/detail?id=2982
+// Our shadows looks wrong. The text not having shadows is to be expected, but
+// it looks like we're missing a red shadow that Safari has.
+// http://code.google.com/p/chromium/issues/detail?id=2982
V8 # LayoutTests/fast/repaint/shadow-multiple-horizontal.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-strict-horizontal.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-strict-vertical.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-vertical.html = FAIL
-# This test was removed from Win, but still fails on Mac
+// This test was removed from Win, but still fails on Mac
V8 # LayoutTests/fast/events/tabindex-focus-blur-all.html = FAIL
-# Regression from the webkit merge submitting forms to reference fragments.
-# http://code.google.com/p/chromium/issues/detail?id=3008
+// Regression from the webkit merge submitting forms to reference fragments.
+// http://code.google.com/p/chromium/issues/detail?id=3008
V8 # LayoutTests/fast/forms/submit-to-url-fragment.html = FAIL
-# Many of the failures here are because V8 stringifies objects differently than
-# JSC. However, the missing position and totalSize properties seems genuine.
+// Many of the failures here are because V8 stringifies objects differently than
+// JSC. However, the missing position and totalSize properties seems genuine.
V8 # SKIP : LayoutTests/fast/dom/xmlhttprequest-get.xhtml = PASS
-# This fails because the CSSParser cannot handle the keyframe rule.
-# Picking up a new parser from webkit should fix the rest.
-V8 # LayoutTests/css3/keyframes-rule.html = FAIL
-
-# We don't render reflection masks properly.
-# http://code.google.com/p/chromium/issues/detail?id=3229
+// We don't render reflection masks properly.
+// http://code.google.com/p/chromium/issues/detail?id=3229
V8 # LayoutTests/fast/reflections/reflection-masks.html = FAIL
-# http://crbug.com/3244 : SVG masks aren't quite working yet.
+// http://crbug.com/3244 : SVG masks aren't quite working yet.
V8 # LayoutTests/fast/backgrounds/svg-as-mask.html = FAIL
V8 # LayoutTests/svg/W3C-SVG-1.1/masking-mask-01-b.svg = FAIL
V8 # LayoutTests/svg/batik/masking/maskRegions.svg = FAIL
-# More tests that fail because of masks not working properly.
+// More tests that fail because of masks not working properly.
V8 # LayoutTests/fast/backgrounds/mask-composite.html = FAIL
V8 # LayoutTests/fast/backgrounds/repeat/mask-negative-offset-repeat.html = FAIL
V8 # LayoutTests/fast/borders/block-mask-overlay-image.html = FAIL
V8 # LayoutTests/fast/borders/inline-mask-overlay-image.html = FAIL
-# This was fixed in WebKit r36103 http://trac.webkit.org/changeset/36103/.
-# For chromium, it got flaky once we pulled 36103 in.
+// This was fixed in WebKit r36103 http://trac.webkit.org/changeset/36103/.
+// For chromium, it got flaky once we pulled 36103 in.
V8 # LayoutTests/svg/custom/invalid-fill-hex.svg = FAIL | PASS
-# This test finishes while in the middle of a CSS animation. This is
-# intentional, but it makes pixel results unreliable.
-# http://bugs.webkit.org/show_bug.cgi?id=21491
+// This test finishes while in the middle of a CSS animation. This is
+// intentional, but it makes pixel results unreliable.
+// http://bugs.webkit.org/show_bug.cgi?id=21491
V8 # SKIP : LayoutTests/transitions/opacity-transition-zindex.html = PASS
-# Issue 3273: TextInputController::firstRectForCharacterRange not implemented
-# http://code.google.com/p/chromium/issues/detail?id=3273
+// Issue 3273: TextInputController::firstRectForCharacterRange not implemented
+// http://code.google.com/p/chromium/issues/detail?id=3273
V8 # LayoutTests/editing/selection/move-left-right.html = FAIL
// ----------------------------------------------------------------------------
diff --git a/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt b/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt
index 76a82cf..61b9b27 100644
--- a/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt
+++ b/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt
@@ -74,7 +74,7 @@ V8 # LayoutTests/http/tests/xmlhttprequest/upload-onload-event.html = FAIL
V8 # LayoutTests/http/tests/xmlhttprequest/upload-onprogress-event.html = FAIL
V8 # LayoutTests/http/tests/xmlhttprequest/upload-progress-events.html = FAIL
-# Bug 1341582
+// Bug 1341582
V8 # LayoutTests/svg/dom/animated-tearoff-equality.xhtml = FAIL | PASS
V8 # LayoutTests/svg/dom/animated-tearoff-lifespan.xhtml = FAIL | PASS
@@ -653,29 +653,29 @@ V8 # pending/http/tests/security/cross-frame-access-protocol.html = PASS | TIMEO
V8 # LayoutTests/fast/js/date-DST-time-cusps.html = FAIL | PASS
V8 # LayoutTests/fast/js/date-big-setdate.html = FAIL | PASS
-# V8 doesn't stable sort. This is not required and may never be fixed. It is
-# tracked by: http://code.google.com/p/v8/issues/detail?id=90
+// V8 doesn't stable sort. This is not required and may never be fixed. It is
+// tracked by: http://code.google.com/p/v8/issues/detail?id=90
V8 # LayoutTests/fast/js/comparefn-sort-stability.html = FAIL
V8 # LayoutTests/fast/js/sort-stability.html = FAIL
-# Apparently V8 date handling:
-# http://code.google.com/p/v8/issues/detail?id=91
+// Apparently V8 date handling:
+// http://code.google.com/p/v8/issues/detail?id=91
V8 # LayoutTests/fast/js/date-set-to-nan.html = FAIL
-# These require application cache to be enabled. Skip the tests because
-# they're all timing out. TODO(tc): Upstream changes to the test so they
-# fail quickly if window.applicationCache is undefined (then we don't have to
-# skip them).
-# http://crbug.com/2844
+// These require application cache to be enabled. Skip the tests because
+// they're all timing out. TODO(tc): Upstream changes to the test so they
+// fail quickly if window.applicationCache is undefined (then we don't have to
+// skip them).
+// http://crbug.com/2844
V8 # SKIP : LayoutTests/http/tests/appcache = TIMEOUT | FAIL
-# Depends on postMessage in a way we don't support:
-# http://code.google.com/p/chromium/issues/detail?id=2857
+// Depends on postMessage in a way we don't support:
+// http://code.google.com/p/chromium/issues/detail?id=2857
V8 # LayoutTests/http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame-uppercase.html = FAIL
V8 # LayoutTests/http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame.html = FAIL
-# V8 failures as a result of the WebKit merge.
-# http://code.google.com/p/v8/issues/detail?id=92
+// V8 failures as a result of the WebKit merge.
+// http://code.google.com/p/v8/issues/detail?id=92
V8 # LayoutTests/fast/js/constructor-attributes.html = FAIL
V8 # LayoutTests/fast/js/eval-cross-window.html = FAIL
V8 # LayoutTests/fast/js/eval-keyword-vs-function.html = FAIL
@@ -689,12 +689,12 @@ V8 # LayoutTests/fast/js/removing-Cf-characters.html = FAIL
V8 # LayoutTests/fast/js/static-scope-object.html = FAIL
V8 # LayoutTests/fast/js/delete-getters-setters.html = FAIL
-# Console output won't have line numbers until V8 gives us a way to get that
-# information. <http://crbug.com/2960>
+// Console output won't have line numbers until V8 gives us a way to get that
+// information. <http://crbug.com/2960>
V8 # LayoutTests/fast/dom/Window/console-functions.html = FAIL
-# Shadows don't render correctly for these tests.
-# http://code.google.com/p/chromium/issues/detail?id=2969
+// Shadows don't render correctly for these tests.
+// http://code.google.com/p/chromium/issues/detail?id=2969
V8 # LayoutTests/fast/canvas/shadow-offset-1.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-2.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-3.html = FAIL
@@ -704,74 +704,70 @@ V8 # LayoutTests/fast/canvas/shadow-offset-6.html = FAIL
V8 # LayoutTests/fast/canvas/shadow-offset-7.html = FAIL
V8 # LayoutTests/fast/css/shadow-multiple.html = FAIL
-# There appears to be some parsing error. These tests give "successfullyParsed
-# should be true (of type boolean). Was undefined (of type undefined)."
-# http://code.google.com/p/chromium/issues/detail?id=2976
+// There appears to be some parsing error. These tests give "successfullyParsed
+// should be true (of type boolean). Was undefined (of type undefined)."
+// http://code.google.com/p/chromium/issues/detail?id=2976
V8 # LayoutTests/fast/canvas/drawImage-with-negative-source-destination.html = FAIL
-# getImageData isn't supported on canvas.
-# http://code.google.com/p/chromium/issues/detail?id=2974
+// getImageData isn't supported on canvas.
+// http://code.google.com/p/chromium/issues/detail?id=2974
V8 # LayoutTests/fast/canvas/canvas-pattern-behaviour.html = FAIL
V8 # LayoutTests/fast/canvas/canvas-getImageData.html = FAIL
-# We don't implement toDataURL.
-# http://code.google.com/p/chromium/issues/detail?id=2972
+// We don't implement toDataURL.
+// http://code.google.com/p/chromium/issues/detail?id=2972
V8 # LayoutTests/fast/canvas/toDataURL-supportedTypes.html = FAIL
-# layoutTestController.setStopProvisionalFrameLoads needs implementing
-# http://code.google.com/p/chromium/issues/detail?id=2980
+// layoutTestController.setStopProvisionalFrameLoads needs implementing
+// http://code.google.com/p/chromium/issues/detail?id=2980
V8 # LayoutTests/fast/loader/stop-provisional-loads.html = FAIL
-# Our shadows looks wrong. The text not having shadows is to be expected, but
-# it looks like we're missing a red shadow that Safari has.
-# http://code.google.com/p/chromium/issues/detail?id=2982
+// Our shadows looks wrong. The text not having shadows is to be expected, but
+// it looks like we're missing a red shadow that Safari has.
+// http://code.google.com/p/chromium/issues/detail?id=2982
V8 # LayoutTests/fast/repaint/shadow-multiple-horizontal.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-strict-horizontal.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-strict-vertical.html = FAIL
V8 # LayoutTests/fast/repaint/shadow-multiple-vertical.html = FAIL
-# Regression from the webkit merge submitting forms to reference fragments.
-# http://code.google.com/p/chromium/issues/detail?id=3008
+// Regression from the webkit merge submitting forms to reference fragments.
+// http://code.google.com/p/chromium/issues/detail?id=3008
V8 # LayoutTests/fast/forms/submit-to-url-fragment.html = FAIL
-# Many of the failures here are because V8 stringifies objects differently than
-# JSC. However, the missing position and totalSize properties seems genuine.
+// Many of the failures here are because V8 stringifies objects differently than
+// JSC. However, the missing position and totalSize properties seems genuine.
V8 # SKIP : LayoutTests/fast/dom/xmlhttprequest-get.xhtml = PASS
-# This fails because the CSSParser cannot handle the keyframe rule.
-# Picking up a new parser from webkit should fix the rest.
-V8 # LayoutTests/css3/keyframes-rule.html = FAIL
-
-# We don't render reflection masks properly.
-# http://code.google.com/p/chromium/issues/detail?id=3229
+// We don't render reflection masks properly.
+// http://code.google.com/p/chromium/issues/detail?id=3229
V8 # LayoutTests/fast/reflections/reflection-masks.html = FAIL
-# http://crbug.com/3244 : SVG masks aren't quite working yet.
+// http://crbug.com/3244 : SVG masks aren't quite working yet.
V8 # LayoutTests/fast/backgrounds/svg-as-mask.html = FAIL
V8 # LayoutTests/svg/W3C-SVG-1.1/masking-mask-01-b.svg = FAIL
V8 # LayoutTests/svg/batik/masking/maskRegions.svg = FAIL
-# More tests that fail because of masks not working properly.
+// More tests that fail because of masks not working properly.
V8 # LayoutTests/fast/backgrounds/mask-composite.html = FAIL
V8 # LayoutTests/fast/backgrounds/repeat/mask-negative-offset-repeat.html = FAIL
V8 # LayoutTests/fast/borders/block-mask-overlay-image.html = FAIL
V8 # LayoutTests/fast/borders/inline-mask-overlay-image.html = FAIL
-# This was fixed in WebKit r36103 http://trac.webkit.org/changeset/36103/.
-# For chromium, it got flaky once we pulled 36103 in.
+// This was fixed in WebKit r36103 http://trac.webkit.org/changeset/36103/.
+// For chromium, it got flaky once we pulled 36103 in.
V8 # LayoutTests/svg/custom/invalid-fill-hex.svg = FAIL | PASS
-# This test finishes while in the middle of a CSS animation. This is
-# intentional, but it makes pixel results unreliable.
-# http://bugs.webkit.org/show_bug.cgi?id=21491
+// This test finishes while in the middle of a CSS animation. This is
+// intentional, but it makes pixel results unreliable.
+// http://bugs.webkit.org/show_bug.cgi?id=21491
V8 # SKIP : LayoutTests/transitions/opacity-transition-zindex.html = PASS
-# Issue 3273: TextInputController::firstRectForCharacterRange not implemented
-# http://code.google.com/p/chromium/issues/detail?id=3273
+// Issue 3273: TextInputController::firstRectForCharacterRange not implemented
+// http://code.google.com/p/chromium/issues/detail?id=3273
V8 # LayoutTests/editing/selection/move-left-right.html = FAIL
-# This test needs to be rebaselined but said rebaselining didn't work and after
-# over an hour wasted, I'm giving up. -brettw
+// This test needs to be rebaselined but said rebaselining didn't work and after
+// over an hour wasted, I'm giving up. -brettw
V8 | KJS # LayoutTests/svg/custom/foreign-object-skew.svg = FAIL
// ----------------------------------------------------------------------------