Bug 29630 - Some layout tests contain tab characters
Summary: Some layout tests contain tab characters
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-22 01:46 PDT by Shinichiro Hamaji
Modified: 2009-09-22 13:49 PDT (History)
1 user (show)

See Also:

Attachments
Patch v1 (10.17 KB, patch)
2009-09-22 01:53 PDT, Shinichiro Hamaji 		sam: review-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shinichiro Hamaji 2009-09-22 01:46:51 PDT 
The following files contain tab characters.

LayoutTests/fast/js/kde/resources/RegExp.js
LayoutTests/fast/js/kde/resources/cast.js
LayoutTests/fast/js/kde/resources/exceptions.js
LayoutTests/fast/js/kde/resources/md5-1.js
LayoutTests/fast/js/kde/resources/statements.js
LayoutTests/fast/js/resources/regexp-extended-characters-crash.js

Patch will come soon.
Comment 1 Shinichiro Hamaji 2009-09-22 01:53:03 PDT 
Created attachment 39909 [details]
Patch v1
Comment 2 Sam Weinig 2009-09-22 07:41:42 PDT 
Comment on attachment 39909 [details]
Patch v1

Tabs in layout tests is fine.  There is no reason to land this change.
Comment 3 Shinichiro Hamaji 2009-09-22 13:49:18 PDT 
(In reply to comment #2)
> (From update of attachment 39909 [details])
> Tabs in layout tests is fine.  There is no reason to land this change.

OK, so, I'll propset allow-tabs for them when I move these files. Thanks for the review!