Bug 33407 - Add layout test for maintaining referrer during redirect
Summary: Add layout test for maintaining referrer during redirect
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nate Chapin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-08 16:07 PST by Nate Chapin
Modified: 2010-01-11 09:30 PST (History)
0 users

See Also:

Attachments
patch (27.43 KB, patch)
2010-01-08 16:12 PST, Nate Chapin 		no flags Details | Formatted Diff | Diff
Attaching the correct patch this time (3.34 KB, patch)
2010-01-08 16:14 PST, Nate Chapin 		dglazkov: review+
Details | Formatted Diff | Diff
Show Obsolete (1) 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 Nate Chapin 2010-01-08 16:07:28 PST 
Original bug: http://code.google.com/p/chromium/issues/detail?id=7357
Fixed in Chromium by: http://trac.webkit.org/changeset/52027 and http://src.chromium.org/viewvc/chrome?view=rev&revision=35379

This bug isn't covered by any layout test, and I think it's a reasonable have a test to ensure document.referrer is properly maintained during a redirect.
Comment 1 Nate Chapin 2010-01-08 16:12:01 PST 
Created attachment 46172 [details]
patch
Comment 2 Nate Chapin 2010-01-08 16:14:59 PST 
Created attachment 46173 [details]
Attaching the correct patch this time
Comment 3 Dimitri Glazkov (Google) 2010-01-08 16:17:32 PST 
Comment on attachment 46173 [details]
Attaching the correct patch this time

layout tests are good for you.
Comment 4 Nate Chapin 2010-01-11 09:30:37 PST 
http://trac.webkit.org/changeset/53081