Bug 29630

Summary:

Some layout tests contain tab characters

Product:

WebKit

Reporter:

Shinichiro Hamaji <hamaji>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WONTFIX

Severity:

Normal

CC:

sam

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Patch v1	sam: review-

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!