Try to fix a finnicky reftest.
authorsenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Dec 2012 17:48:34 +0000 (17:48 +0000)
committersenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 9 Dec 2012 17:48:34 +0000 (17:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=104486

Reviewed by Simon Fraser.

This reftest fails on Chromium Mac since the leftmost pixel of the 'T'
of 'Text' is clipped in composited mode, and non-clipped in software
painting.  Not sure exactly why, but a lowercase 't' doesn't seem to
have this problem.

* compositing/background-color/background-color-change-to-text-expected.html:
* compositing/background-color/background-color-change-to-text.html:
* platform/chromium-mac/compositing/background-color/background-color-change-to-text-expected.png: Removed.
* platform/chromium-mac/platform/chromium/virtual/softwarecompositing/background-color/background-color-change-to-text-expected.png: Removed.
* platform/chromium/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@137079 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/compositing/background-color/background-color-change-to-text-expected.html
LayoutTests/compositing/background-color/background-color-change-to-text.html
LayoutTests/platform/chromium/TestExpectations

index 25750a5..c525cdd 100644 (file)
@@ -1,5 +1,23 @@
 2012-12-09  Stephen White  <senorblanco@chromium.org>
 
+        Try to fix a finnicky reftest.
+        https://bugs.webkit.org/show_bug.cgi?id=104486
+
+        Reviewed by Simon Fraser.
+
+        This reftest fails on Chromium Mac since the leftmost pixel of the 'T'
+        of 'Text' is clipped in composited mode, and non-clipped in software
+        painting.  Not sure exactly why, but a lowercase 't' doesn't seem to
+        have this problem.
+
+        * compositing/background-color/background-color-change-to-text-expected.html:
+        * compositing/background-color/background-color-change-to-text.html:
+        * platform/chromium-mac/compositing/background-color/background-color-change-to-text-expected.png: Removed.
+        * platform/chromium-mac/platform/chromium/virtual/softwarecompositing/background-color/background-color-change-to-text-expected.png: Removed.
+        * platform/chromium/TestExpectations:
+
+2012-12-09  Stephen White  <senorblanco@chromium.org>
+
         [Chromium] Unreviewed gardening.
 
         * platform/chromium-linux-x86/platform/chromium/virtual/softwarecompositing/rtl/rtl-fixed-overflow-scrolled-expected.txt: Added.
index ec5669a..6ef6d12 100644 (file)
@@ -24,7 +24,7 @@
 
         window.setTimeout(function() {
             // Change the layer to become background only.
-            bg.innerHTML = "Text";
+            bg.innerHTML = "text";
             if (window.testRunner) {
                 window.setTimeout(function() {
                     testRunner.notifyDone();
index c22b2de..2215b95 100644 (file)
@@ -24,7 +24,7 @@
 
         window.setTimeout(function() {
             // Change the layer to become background only.
-            bg.innerHTML = "Text";
+            bg.innerHTML = "text";
             if (window.testRunner) {
                 window.setTimeout(function() {
                     testRunner.notifyDone();
index f948c3d..a19095a 100644 (file)
@@ -4230,10 +4230,6 @@ Bug(senorblanco) [ Win ] http/tests/w3c/webperf/submission/Intel/user-timing/tes
 webkit.org/b/94002 [ Win Mac Debug ] inspector/debugger/live-edit-breakpoints.html [ Failure Timeout ]
 webkit.org/b/94002 [ Mac Debug ] inspector/debugger/live-edit.html [ Crash Pass Timeout ]
 
-# This one has correct results, but the tools refuse to rebaseline it.
-Bug(senorblanco) [ MountainLion ] compositing/background-color/background-color-change-to-text.html [ ImageOnlyFailure ]
-Bug(senorblanco) [ MountainLion ] platform/chromium/virtual/softwarecompositing/background-color/background-color-change-to-text.html [ ImageOnlyFailure ]
-
 # Asserting since r136948.
 webkit.org/b/104473 [ Debug ] inspector/styles/import-pseudoclass-crash.html [ Crash ]