Bug 12792

Summary: [S60] PNIO-6YFCUT: Strange conversion problem (in get " changed to %2522 instead of %22) with for example google search.
Product: WebKit Reporter: Wei Liu <wei.liu>
Component: Page LoadingAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Blocker CC: bradley.morrison
Priority: P1    
Version: 420+   
Hardware: S60 Emulator   
OS: S60 3rd edition   
URL: http://www.google.com
Attachments:
Description Flags
Fix on 3.2 with UrlRequestInfo.cpp
none
updated 3.2 patch zalan: review+

Description Wei Liu 2007-02-16 11:19:43 PST
- Steps to reproduce:
1. Go to www.google.com
2. Search for "Masa Mainio"

- Actual outcome:

Results for %22Masa Mainio%22

- Expected outcome:

Results for "Masa Mainio"

- Other info:

So corresponding gets are like this:
Correct (SPP wk04): GET /search?hl=en&q=%22masa+mainio%22&btnG=Google+Search HTTP/1.1
Incorrect (S60 wk06): GET /search?hl=en&q=%2522masa+mainio%2522&btnG=Google+Search HTTP/1.1
Comment 1 Wei Liu 2007-02-16 11:26:46 PST
Created attachment 13203 [details]
Fix on 3.2 with UrlRequestInfo.cpp
Comment 2 Wei Liu 2007-02-16 11:40:18 PST
Created attachment 13204 [details]
updated 3.2 patch
Comment 3 Wei Liu 2007-02-16 11:42:28 PST
Comment on attachment 13204 [details]
updated 3.2 patch

Please merge the changes to 3.1m branch except for the changes in xmlhttprequest.cpp, that change is for 3.2 only.
Comment 4 Wei Liu 2007-02-16 11:42:30 PST
Comment on attachment 13204 [details]
updated 3.2 patch

Please merge the changes to 3.1m branch except for the changes in xmlhttprequest.cpp, that change is for 3.2 only.
Comment 5 Wei Liu 2007-02-16 11:50:49 PST
Please merge the 3.2 patch to 3.1m, except for xmlhttprequest.cpp change since that is not needed for 3.1m.
Comment 6 zalan 2007-02-16 11:54:12 PST
Comment on attachment 13204 [details]
updated 3.2 patch

r=me
Comment 7 Bradley Morrison 2007-02-16 15:33:34 PST
(In reply to comment #6)
> (From update of attachment 13204 [details] [edit])
> r=me
> 

landed in trunk: r19669 and 3.1: 19664
Comment 8 Krishna 2007-02-25 21:16:33 PST
Verified and Closed.
Comment 9 Joel Parks 2011-03-21 11:53:11 PDT
re-purposing InTSW keyword for use by QtWebkit team