Bug 44033

Summary:

Some http/tests/history tests now pass on Chromium after r65340

Product:

WebKit

Reporter:

Mihai Parparita <mihaip>

Component:

Tools / Tests

Assignee:

Mihai Parparita <mihaip>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, japhet

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none

Mihai Parparita

Reported 2010-08-15 10:03:47 PDT

From the canary bot output: Expected to fail, but passed: (5) http/tests/history/redirect-200-refresh-0-seconds.pl http/tests/history/redirect-js-location-replace-0-seconds.html http/tests/history/redirect-js-location-replace-2-seconds.html http/tests/history/redirect-js-location-replace-before-load.html http/tests/history/redirect-meta-refresh-0-seconds.html See https://bugs.webkit.org/show_bug.cgi?id=42861#c32 for details as to why they pass now. These tests were initially added to the "fixable" list with http://crrev.com/8946, and then to the "ignore" list with http://crrev.com/9287, so there was a bug filed for them.

Attachments
Patch (2.07 KB, patch) 2010-08-15 11:45 PDT, Mihai Parparita	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Mihai Parparita

Comment 1 2010-08-15 11:45:32 PDT

Created attachment 64451 [details] Patch

Mihai Parparita

Comment 2 2010-08-15 11:49:36 PDT

Nate, as the gardener/reviewer, do you mind reviewing this? (in the description I meant to say "so there _never_ was a bug filed for them")

Nate Chapin

Comment 3 2010-08-16 11:28:15 PDT

Comment on attachment 64451 [details] Patch Yay!

WebKit Commit Bot

Comment 4 2010-08-16 12:00:55 PDT

Comment on attachment 64451 [details] Patch Clearing flags on attachment: 64451 Committed r65444: <http://trac.webkit.org/changeset/65444>

WebKit Commit Bot

Comment 5 2010-08-16 12:01:01 PDT

All reviewed patches have been landed. Closing bug.

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