Bug 12269

Summary: [S60] Browser - Form/The TC is not displayed, it's a white page displayed
Product: WebKit Reporter: Shyam Sareen <shyam.sareen>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal CC: koivisto, shyam.sareen
Priority: P2    
Version: 420+   
Hardware: S60 Emulator   
OS: S60 3rd edition   
Attachments:
Description Flags
Patch for 3.2
bradley.morrison: review+
Patch for 3.1
none
Correct the spacing for 3.1m. koivisto: review+

Shyam Sareen
Reported 2007-01-14 13:43:26 PST
TSW ID: MPAK-6XBAA7. ============================== Simple test case: <html> <head> <title/> </head> <body> Hi </body> </html> ================================== The above is displayed with Firefox, Opera and old S60 browser. But, this can't be displayed with the Safari and current S60 browser. This should be displayed with current S60 browser. The problem in the <title/> tag. The <title/> syntax is incorrect.
Attachments
Patch for 3.2 (3.19 KB, patch)
2007-01-14 14:00 PST, Shyam Sareen 		bradley.morrison: review+
DetailsFormatted DiffDiff
Patch for 3.1 (3.04 KB, text/plain)
2007-01-14 14:01 PST, Shyam Sareen 		no flags
Details
Correct the spacing for 3.1m. (3.08 KB, patch)
2007-01-15 07:06 PST, Shyam Sareen 		koivisto: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Shyam Sareen
Comment 1 2007-01-14 14:00:15 PST
Created attachment 12432 [details] Patch for 3.2
Shyam Sareen
Comment 2 2007-01-14 14:01:07 PST
Created attachment 12433 [details] Patch for 3.1
Antti Koivisto
Comment 3 2007-01-15 06:04:00 PST
Is this problem also visible in TOT? Looks ok, except identation is wrong here: if(len > 1 && ptr[len-1] == '/' ) + { +#ifdef NOKIA_CHANGES + slashAtEndOfTag = true; +#endif ptr[--len] = '\0'; + }
Shyam Sareen
Comment 4 2007-01-15 07:06:39 PST
Created attachment 12455 [details] Correct the spacing for 3.1m. The problem is visible in the CCB/3.2 also. That is reason, I have attached two patch. I verfied on Mac Safari, the problem is visible there also. The spacing seem correct for 3.2 patch.
Antti Koivisto
Comment 5 2007-01-18 03:00:02 PST
r=me
Yongjun Zhang
Comment 6 2007-01-18 08:34:35 PST
3.1m fix completed: At revision: 18938
Bradley Morrison
Comment 7 2007-01-18 08:46:22 PST
Comment on attachment 12432 [details] Patch for 3.2 Clearing review flag
Yongjun Zhang
Comment 8 2007-01-18 08:48:41 PST
Comment on attachment 12432 [details] Patch for 3.2 r = me
Yongjun Zhang
Comment 9 2007-01-18 08:50:58 PST
3.1m fix completed: At revision: 18939
Krishna
Comment 10 2007-02-09 13:25:02 PST
Verified and Closed.
Note You need to log in before you can comment on or make changes to this bug.