diff options
author | tc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-20 22:54:35 +0000 |
---|---|---|
committer | tc@google.com <tc@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-20 22:54:35 +0000 |
commit | b93568f763472809870ff37735eab7e9cfcdb3c8 (patch) | |
tree | 14a491288fe99bed1daa6835f71219b2c0de8d1d | |
parent | acfc5091b264cd786a3dc656977ce7451aea6e88 (diff) | |
download | chromium_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
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 Binary files differindex 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 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 Binary files differnew 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 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 Binary files differindex 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 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 Binary files differnew file mode 100755 index 0000000..023615e --- /dev/null +++ b/webkit/tools/test_shell/resources/textAreaResizeCorner.png 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; |