Summary: | [S60] EJGO-7458XN: document.write in href of <a> doesn't display correctly | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Pat <vincent.bradley> | ||||||||
Component: | DOM | Assignee: | Pat <vincent.bradley> | ||||||||
Status: | CLOSED FIXED | ||||||||||
Severity: | Critical | CC: | cdumez | ||||||||
Priority: | P2 | Keywords: | HasReduction, PlatformOnly | ||||||||
Version: | 523.x (Safari 3) | ||||||||||
Hardware: | S60 Emulator | ||||||||||
OS: | S60 3rd edition | ||||||||||
URL: | http://waplabdc.nokia-boston.com/browser/users/S60_BrowserNG/S60NG_ScalableUI/S60NG_ScalableUI_MarkUP/ecma_prompt.html | ||||||||||
Attachments: |
|
Description
Pat
2007-08-06 07:45:50 PDT
Created attachment 15849 [details]
test case reduction
test case reduction
Created attachment 15851 [details] fix for <a> with javascript in href attribute If we have javascript" in the anchor tag's href <a href="javascript: ...", clear the off-screen bitmap before drawing. Some additional test cases were tested: http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt1.html http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt2.html http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt3.html http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt4.html http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt5.html http://waplabdc.nokia-boston.com/browser/users/vbradley/bugs_ecma_prompt/ecma_prompt6.html Review requested from Zalan, as per Sachin. Zalan is also aware of issue and has TOT to test on. This fix is for 3.2 only. Created attachment 15852 [details]
3.2 patch for javascript in <a> href
2nd patch for 3.2
Comment on attachment 15852 [details]
3.2 patch for javascript in <a> href
I think the patch is incorrect (seems like a hack to my opinion), please figure out the real reason why page opened with javascript: links doesn't refresh the screen.
(In reply to comment #5) > (From update of attachment 15852 [details] [edit]) > I think the patch is incorrect (seems like a hack to my opinion), please figure > out the real reason why page opened with javascript: links doesn't refresh the > screen. Checked in fix for this bug. Used the document.write bug http://bugs.webkit.org/show_bug.cgi?id=14915 to submit the patch. Bulk closing of all s60 platform bugs. Sorry for the noise! re-purposing InTSW keyword for use by QtWebkit team Mass moving XML DOM bugs to the "DOM" Component. |