Bug 74000 - [Qt] HTTP redirect fails when irrelevant but incorrect encoding is specified
Summary: [Qt] HTTP redirect fails when irrelevant but incorrect encoding is specified
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-07 07:36 PST by Allan Sandfeld Jensen
Modified: 2011-12-08 12:34 PST (History)
2 users (show)

See Also:


Attachments
Patch (2.11 KB, patch)
2011-12-07 07:48 PST, Allan Sandfeld Jensen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Allan Sandfeld Jensen 2011-12-07 07:36:59 PST
Originally reported as NB#261819:

PRECONDITIONS: 
Yandex set as a default search engine.

STEPS LEADING TO PROBLEM: 
1. Open Browser
2. Type e.g. 'car' into URL field and perform search
3. Perform another search (e.g. 'bad cars')
4. Repeat step 3. several times

EXPECTED OUTCOME:
Search i always performed properly.

ACTUAL OUTCOME:
At some point dialog pops up: "Cannot open page; Found" 
PRECONDITIONS: 
Yandex set as a default search engine.

STEPS LEADING TO PROBLEM: 
1. Open Browser
2. Type e.g. 'car' into URL field and perform search
3. Perform another search (e.g. 'bad cars')
4. Repeat step 3. several times

EXPECTED OUTCOME:
Search i always performed properly.

ACTUAL OUTCOME:
At some point dialog pops up: "Cannot open page; Found"
Comment 1 Allan Sandfeld Jensen 2011-12-07 07:48:00 PST
Created attachment 118211 [details]
Patch
Comment 2 Alexey Proskuryakov 2011-12-07 10:10:38 PST
Comment on attachment 118211 [details]
Patch

Please add a regression test.
Comment 3 Simon Hausmann 2011-12-07 11:12:06 PST
I agree with AP. It would be great if you could add a test or maybe you can find that there exists a test already that we are skipping and that can be unskipped with your patch. I suspect it would be somewhere in LayoutTests/http.

Adding test coverage would be a great way to complete Kimmo's original work.
Comment 4 Allan Sandfeld Jensen 2011-12-08 03:43:06 PST
Additional testing reveals the issue has been resolved on a different level in trunk. This patch in platform is no longer necessary