Bug 14436 - [S60][3.2] TMCN-744MYN - Browser not supporting Data URL
Summary: [S60][3.2] TMCN-744MYN - Browser not supporting Data URL
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Emulator S60 3rd edition
: P2 Normal
Assignee: Vikram
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-27 13:47 PDT by Vikram
Modified: 2008-04-09 11:39 PDT (History)
0 users

See Also:

Attachments
fix (2.41 KB, patch)
2007-06-27 13:59 PDT, Vikram 		Sachin.Padma: review-
Details | Formatted Diff | Diff
revised patch (2.36 KB, patch)
2007-06-28 14:33 PDT, Vikram 		no flags Details | Formatted Diff | Diff
revised patch (2.52 KB, patch)
2007-06-29 07:45 PDT, Vikram 		Sachin.Padma: review+
Details | Formatted Diff | Diff
Show Obsolete (2) 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 Vikram 2007-06-27 13:47:33 PDT 
TSW ID:  TMCN-744MYN 
The browser does not render any content from a data:URL
Comment 1 Vikram 2007-06-27 13:59:19 PDT 
Created attachment 15277 [details]
fix
Comment 2 Sachin Padma 2007-06-28 11:41:08 PDT 
Comment on attachment 15277 [details]
fix

No need to allocate the iBody when doing EscapeUtils::EscapeDecodeL. You are introducing a memory leak. Also why is return code from codec.Decode being ignored?
Comment 3 Vikram 2007-06-28 14:33:39 PDT 
Created attachment 15298 [details]
revised patch
Comment 4 Vikram 2007-06-29 07:45:07 PDT 
Created attachment 15315 [details]
revised patch
Comment 5 Sachin Padma 2007-06-29 13:28:26 PDT 
Comment on attachment 15315 [details]
revised patch

r=me
Comment 6 Sachin Padma 2007-06-29 13:29:43 PDT 
landed in r23887
Comment 7 Bradley Morrison 2008-04-09 11:39:44 PDT 
Bulk closing of all s60 platform bugs. 

Sorry for the noise!