Summary: | REGRESSION (Safari 4.0.4): HTTP 307 after a 303 after a POST re-sends POST data from the original request (31410) | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | masterchiff | ||||||
Component: | Page Loading | Assignee: | Brady Eidson <beidson> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, beidson, emacemac7, eric, mjs, simon.fraser, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
masterchiff
2009-11-12 08:04:32 PST
Does this affect one or more public website? I have a fix upcoming... Note due to the design of our networking layer abstraction, I've only been able to fix this on Mac and Windows. Other platforms are possibly/probably affected. Created attachment 49252 [details]
Mac/Win fix + layouttest
Attachment 49252 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/platform/network/cf/ResourceHandleCFNet.cpp:435: Tab found; better to use spaces [whitespace/tab] [1]
Total errors found: 1 in 14 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 49253 [details]
Fix the tab, no other changes
Comment on attachment 49253 [details]
Fix the tab, no other changes
r=me
I'm not sure if quietly disabling the test for other problem is best - maybe you should leave it fail for platform maintainers to decide what to do, or just go ahead and file bugs for each.
(In reply to comment #8) > (From update of attachment 49253 [details]) > r=me > > I'm not sure if quietly disabling the test for other problem is best - maybe > you should leave it fail for platform maintainers to decide what to do, or just > go ahead and file bugs for each. It's generally our policy to not knowingly introduce layouttest failures to another platform, isn't it? I'll file the bugs. Filed https://bugs.webkit.org/show_bug.cgi?id=35300 and https://bugs.webkit.org/show_bug.cgi?id=35301 for gtk and qt to follow up on this. Note Chromium should probably follow up on this too, but they don't even seem to have a Skipped list for me to follow, so... Looks like this was landed: http://trac.webkit.org/changeset/55157 |