Bug 12951

Summary:

REGRESSION (r19949): LayoutTests/fast/xsl/xslt-processer.html fails

Product:

WebKit

Reporter:

David Kilzer (:ddkilzer) <ddkilzer>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

justin.garcia

Priority:

Keywords:

LayoutTestFailure, Regression

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

URL:

http://build.webkit.org/results/post-commit-powerpc-mac-os-x/5863/

Attachments:

Description	Flags
Patch v1	hyatt: review-
Patch v2	zimmermann: review+

David Kilzer (:ddkilzer)

Reported 2007-03-02 17:02:10 PST

The results for LayoutTests/fast/xsl/xslt-processer.html broke (progressed) with r19949, the fix for Bug 12735. Patch forthcoming.

Attachments
Patch v1 (1.32 KB, patch) 2007-03-02 17:16 PST, David Kilzer (:ddkilzer)	hyatt: review-	Details Formatted Diff Diff
Patch v2 (33.66 KB, patch) 2007-03-02 17:57 PST, David Kilzer (:ddkilzer)	zimmermann: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

David Kilzer (:ddkilzer)

Comment 1 2007-03-02 17:16:06 PST

Created attachment 13459 [details] Patch v1 Proposed patch.

Dave Hyatt

Comment 2 2007-03-02 17:30:13 PST

Comment on attachment 13459 [details] Patch v1 Please move the layout test to fix the spelling error. It should be processor not processer.

David Kilzer (:ddkilzer)

Comment 3 2007-03-02 17:57:08 PST

Created attachment 13460 [details] Patch v2 Fixed misspelling of "processer" to "processor" per Comment #2.

Nikolas Zimmermann

Comment 4 2007-03-04 03:52:04 PST

Comment on attachment 13460 [details] Patch v2 r=me. Please use "svn mv" to preserve history when moving the actual test file (processer -> processor).

Sam Weinig

Comment 5 2007-03-04 08:02:45 PST

Landed in r19953.

David Kilzer (:ddkilzer)

Comment 6 2007-03-04 08:03:20 PST

Committed revision 19953. I also had to set the allow-tabs property on these files: LayoutTests/fast/xsl/xslt-processor-expected.txt LayoutTests/fast/xsl/xslt-processor.html

Note You need to log in before you can comment on or make changes to this bug.