RESOLVED FIXED 91806
[Tests] Rename fast/forms/number/*.html to number-*.html 
https://bugs.webkit.org/show_bug.cgi?id=91806
Summary [Tests] Rename fast/forms/number/*.html to number-*.html
yosin
Reported 2012-07-19 20:56:03 PDT
This bug is a part of fast/forms arrangement.
Attachments
Patch 1 (718.06 KB, patch)
2012-07-20 02:07 PDT, yosin 		tkent: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
yosin
Comment 1 2012-07-20 02:07:23 PDT
Created attachment 153451 [details] Patch 1
yosin
Comment 2 2012-07-20 02:09:35 PDT
Comment on attachment 153451 [details] Patch 1 Could you review this patch? Thanks in advance. This is result of "svn move". So, I'll commit from terminal.
Kent Tamura
Comment 3 2012-07-20 02:51:10 PDT
Will you rename LayoutTests/platform/mac/fast/forms/input-appearance-spinbutton*.html in another patch?
yosin
Comment 4 2012-07-20 02:57:06 PDT
(In reply to comment #3) > Will you rename LayoutTests/platform/mac/fast/forms/input-appearance-spinbutton*.html in another patch? It is out of scope of this bug. This bug is intended for renaming files under LayoutTests/fast/form/numbers/.
yosin
Comment 5 2012-07-20 03:03:27 PDT
(In reply to comment #3) > Will you rename LayoutTests/platform/mac/fast/forms/input-appearance-spinbutton*.html in another patch? In the future, yes. There are 13 files under LayoutTest/platform/*/fast/forms/.
Kent Tamura
Comment 6 2012-07-20 03:45:21 PDT
Comment on attachment 153451 [details] Patch 1 rubber stamped. Please watch the buildbots for a while after landing this patch.
yosin
Comment 7 2012-07-22 18:05:37 PDT
Renaming is succeeded: 1. Change set displays entries with "moved from" 2. Individual file entry has "copied from", e.g. http://trac.webkit.org/log/trunk/LayoutTests/fast/forms/number/number-size.html?rev=123302 I'm watching "Waterfall"
Andrew Wilson
Comment 8 2012-07-23 10:55:07 PDT
I'm seeing these errors in the webkit lint output: e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:228 Path does not exist. fast/forms/number/input-appearance-number-rtl.html e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:229 Path does not exist. fast/forms/number/input-appearance-spinbutton-disabled-readonly.html e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:230 Path does not exist. fast/forms/number/input-appearance-spinbutton-layer.html
yosin
Comment 9 2012-07-23 19:16:37 PDT
(In reply to comment #8) > I'm seeing these errors in the webkit lint output: > > e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:228 Path does not exist. fast/forms/number/input-appearance-number-rtl.html > e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:229 Path does not exist. fast/forms/number/input-appearance-spinbutton-disabled-readonly.html > e:\b\build\slave\Webkit_Win7\build\src\skia\skia_test_expectations.txt:230 Path does not exist. fast/forms/number/input-appearance-spinbutton-layer.html Thanks Andrew! I updates skia_test_expectations.txt http://codereview.chromium.org/10810065
Note You need to log in before you can comment on or make changes to this bug.