summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-20 22:54:35 +0000
committertc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2008-11-20 22:54:35 +0000
commitb93568f763472809870ff37735eab7e9cfcdb3c8 (patch)
tree14a491288fe99bed1daa6835f71219b2c0de8d1d
parentacfc5091b264cd786a3dc656977ce7451aea6e88 (diff)
downloadchromium_src-b93568f763472809870ff37735eab7e9cfcdb3c8.zip
chromium_src-b93568f763472809870ff37735eab7e9cfcdb3c8.tar.gz
chromium_src-b93568f763472809870ff37735eab7e9cfcdb3c8.tar.bz2
Fix layout tests that regressed by changing the resize corner. These tests needed a new image baseline.
The last webkit merge also pulled in a new resize corner test. It's passing except for font differences to I rebaselined that as well. Review URL: http://codereview.chromium.org/11549 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5795 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.checksum2
-rw-r--r--webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.pngbin5157 -> 5055 bytes
-rwxr-xr-xwebkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.checksum1
-rwxr-xr-xwebkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.pngbin0 -> 7540 bytes
-rwxr-xr-xwebkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.txt21
-rw-r--r--webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.checksum2
-rw-r--r--webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.pngbin7074 -> 7107 bytes
-rw-r--r--webkit/tools/layout_tests/test_lists/win/tests_fixable.txt4
-rwxr-xr-xwebkit/tools/test_shell/resources/textAreaResizeCorner.pngbin0 -> 195 bytes
-rw-r--r--webkit/tools/test_shell/test_shell.cc54
10 files changed, 60 insertions, 24 deletions
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.checksum b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.checksum
index ab7ca0d..383821c 100644
--- a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.checksum
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.checksum
@@ -1 +1 @@
-37807b616e5fcd0c1bcf181d4ed9eb1e \ No newline at end of file
+beea79932ae32d753696430fb674ddaa \ No newline at end of file
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.png b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.png
index 3978ade8..bcd2ecb 100644
--- a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.png
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-expected.png
Binary files differ
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.checksum b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.checksum
new file mode 100755
index 0000000..3fd0f29
--- /dev/null
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.checksum
@@ -0,0 +1 @@
+f524c01d70124c24f5f798737fb46c12 \ No newline at end of file
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.png b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.png
new file mode 100755
index 0000000..118479e
--- /dev/null
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.png
Binary files differ
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.txt b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.txt
new file mode 100755
index 0000000..01e3c0f
--- /dev/null
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/css/resize-corner-tracking-transformed-expected.txt
@@ -0,0 +1,21 @@
+layer at (0,0) size 800x600
+ RenderView at (0,0) size 800x600
+layer at (0,0) size 800x600
+ RenderBlock {HTML} at (0,0) size 800x600
+ RenderBody {BODY} at (8,8) size 784x542
+ RenderBlock {HR} at (0,0) size 784x2 [border: (1px inset #000000)]
+ RenderBlock {P} at (0,18) size 784x20
+ RenderText {#text} at (0,0) size 51x19
+ text run at (0,0) width 51: "Test for "
+ RenderInline {I} at (0,0) size 655x19
+ RenderInline {A} at (0,0) size 311x19 [color=#0000EE]
+ RenderText {#text} at (51,0) size 311x19
+ text run at (51,0) width 311: "https://bugs.webkit.org/show_bug.cgi?id=22118"
+ RenderText {#text} at (362,0) size 344x19
+ text run at (362,0) width 4: " "
+ text run at (366,0) width 340: "Resize corner does not track in transformed elements"
+ RenderText {#text} at (706,0) size 4x19
+ text run at (706,0) width 4: "."
+ RenderBlock {HR} at (0,54) size 784x2 [border: (1px inset #000000)]
+layer at (58,114) size 163x129 clip at (60,116) size 159x125
+ RenderBlock {DIV} at (50,106) size 163x129 [border: (2px solid #0000FF)]
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.checksum b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.checksum
index d8123e2..8d36e23 100644
--- a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.checksum
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.checksum
@@ -1 +1 @@
-864cdf598f29878e01678ffb893f2d8e \ No newline at end of file
+7ed8c9d9a35cdb0645611611b672dd92 \ No newline at end of file
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.png b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.png
index 48712dd..90eacd1 100644
--- a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.png
+++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/overflow/hit-test-overflow-controls-expected.png
Binary files differ
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 f3381f8..0e471e9 100644
--- a/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt
+++ b/webkit/tools/layout_tests/test_lists/win/tests_fixable.txt
@@ -1112,7 +1112,3 @@ LayoutTests/svg/W3C-SVG-1.1/text-path-01-b.svg = FAIL
// Started failing after http://codereview.chromium.org/11205
LayoutTests/svg/W3C-SVG-1.1/animate-elem-02-t.svg = FAIL
-
-// TODO(tc): Regressed when changing the resize corner.
-LayoutTests/fast/css/resize-corner-tracking.html = FAIL
-LayoutTests/fast/overflow/hit-test-overflow-controls.html = FAIL
diff --git a/webkit/tools/test_shell/resources/textAreaResizeCorner.png b/webkit/tools/test_shell/resources/textAreaResizeCorner.png
new file mode 100755
index 0000000..023615e
--- /dev/null
+++ b/webkit/tools/test_shell/resources/textAreaResizeCorner.png
Binary files differ
diff --git a/webkit/tools/test_shell/test_shell.cc b/webkit/tools/test_shell/test_shell.cc
index 02d268c..04b05b3 100644
--- a/webkit/tools/test_shell/test_shell.cc
+++ b/webkit/tools/test_shell/test_shell.cc
@@ -42,6 +42,8 @@
#include "SkBitmap.h"
+namespace {
+
// Default timeout for page load when running non-interactive file
// tests, in ms.
const int kDefaultFileTestTimeoutMillisecs = 10 * 1000;
@@ -55,11 +57,15 @@ const int kTestWindowHeight = 600;
const int kSVGTestWindowWidth = 480;
const int kSVGTestWindowHeight = 360;
-// Initialize static member variable
-WindowList* TestShell::window_list_;
-WebPreferences* TestShell::web_prefs_ = NULL;
-bool TestShell::interactive_ = true;
-int TestShell::file_test_timeout_ms_ = kDefaultFileTestTimeoutMillisecs;
+// Helper method for getting the path to the test shell resources directory.
+FilePath GetResourcesFilePath() {
+ FilePath path;
+ PathService::Get(base::DIR_SOURCE_ROOT, &path);
+ path = path.Append(FILE_PATH_LITERAL("webkit"));
+ path = path.Append(FILE_PATH_LITERAL("tools"));
+ path = path.Append(FILE_PATH_LITERAL("test_shell"));
+ return path.Append(FILE_PATH_LITERAL("resources"));
+}
// URLRequestTestShellFileJob is used to serve the inspector
class URLRequestTestShellFileJob : public URLRequestFileJob {
@@ -85,6 +91,14 @@ class URLRequestTestShellFileJob : public URLRequestFileJob {
DISALLOW_COPY_AND_ASSIGN(URLRequestTestShellFileJob);
};
+} // namespace
+
+// Initialize static member variable
+WindowList* TestShell::window_list_;
+WebPreferences* TestShell::web_prefs_ = NULL;
+bool TestShell::interactive_ = true;
+int TestShell::file_test_timeout_ms_ = kDefaultFileTestTimeoutMillisecs;
+
TestShell::TestShell()
: m_mainWnd(NULL),
m_editWnd(NULL),
@@ -447,12 +461,7 @@ std::string GetDataResource(int resource_id) {
// Use webkit's broken image icon (16x16)
static std::string broken_image_data;
if (broken_image_data.empty()) {
- FilePath path;
- PathService::Get(base::DIR_SOURCE_ROOT, &path);
- path = path.Append(FILE_PATH_LITERAL("webkit"));
- path = path.Append(FILE_PATH_LITERAL("tools"));
- path = path.Append(FILE_PATH_LITERAL("test_shell"));
- path = path.Append(FILE_PATH_LITERAL("resources"));
+ FilePath path = GetResourcesFilePath();
path = path.Append(FILE_PATH_LITERAL("missingImage.gif"));
bool success = file_util::ReadFileToString(path.ToWStringHack(),
&broken_image_data);
@@ -468,16 +477,11 @@ std::string GetDataResource(int resource_id) {
// that computes feed previews in feed_preview.cc:MakeFeedPreview.
// This fixes issue #932714.
return std::string("Feed preview for {{URL}}");
- case IDR_EDITOR_DELETE_BUTTON:{
+ case IDR_EDITOR_DELETE_BUTTON: {
// Use webkit's delete button image.
static std::string delete_button_data;
if (delete_button_data.empty()) {
- FilePath path;
- PathService::Get(base::DIR_SOURCE_ROOT, &path);
- path = path.Append(FILE_PATH_LITERAL("webkit"));
- path = path.Append(FILE_PATH_LITERAL("tools"));
- path = path.Append(FILE_PATH_LITERAL("test_shell"));
- path = path.Append(FILE_PATH_LITERAL("resources"));
+ FilePath path = GetResourcesFilePath();
path = path.Append(FILE_PATH_LITERAL("deleteButton.png"));
bool success = file_util::ReadFileToString(path.ToWStringHack(),
&delete_button_data);
@@ -487,6 +491,20 @@ std::string GetDataResource(int resource_id) {
}
return delete_button_data;
}
+ case IDR_TEXTAREA_RESIZER: {
+ // Use webkit's text area resizer image.
+ static std::string resize_corner_data;
+ if (resize_corner_data.empty()) {
+ FilePath path = GetResourcesFilePath();
+ path = path.Append(FILE_PATH_LITERAL("textAreaResizeCorner.png"));
+ bool success = file_util::ReadFileToString(path.ToWStringHack(),
+ &resize_corner_data);
+ if (!success) {
+ LOG(FATAL) << "Failed reading: " << path.value();
+ }
+ }
+ return resize_corner_data;
+ }
default:
break;