| Summary: | [S60] File transaction should handle the case of escaped file schemed url | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Wei Liu <wei.liu> | ||||||||||
| Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> | ||||||||||
| Status: | CLOSED FIXED | ||||||||||||
| Severity: | Major | ||||||||||||
| Priority: | P3 | ||||||||||||
| Version: | 420+ | ||||||||||||
| Hardware: | S60 Hardware | ||||||||||||
| OS: | S60 3rd edition | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Wei Liu
2006-10-18 10:37:44 PDT
Created attachment 11136 [details]
file transaction fix for escaped file url
Comment on attachment 11136 [details]
file transaction fix for escaped file url
The escaping should be generic.
Created attachment 11258 [details]
Files: BrCtl.cpp, FileTransaction.cpp
This patch uses the TUriParser::GetFileNameL(EUriFileNameFull) to generate the file path.
Created attachment 11287 [details]
Files: BrCtl.cpp, FileTransaction.cpp
Added a TRAP_IGNORE around TUriParser::GetFileNameL since invalid paths should return NULL and are not expected to leave.
Created attachment 11330 [details]
Files: FileTransaction.cpp
Remove any escaped characters in the file path with EscapeUtils::EscapeDecodeL.
Comment on attachment 11330 [details]
Files: FileTransaction.cpp
r=me
|