Bug 14888

Summary: [S60] EJGO-7458XN: document.write in href of <a> doesn't display correctly
Product: WebKit Reporter: Pat <vincent.bradley>
Component: DOMAssignee: 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 Flags
test case reduction
none
fix for <a> with javascript in href attribute
none
3.2 patch for javascript in <a> href yongjun.zhang: review-

Description Pat 2007-08-06 07:45:50 PDT
TSW ID EJGO-7458XN
Web contents are overlapped together after accepting the query.

<html>
<head>
<script type="text/javascript">
function message(){
document.write("Hello John Wayne");
document.write("</body></html>");
}
</script>
</head>
<body>
<p> Select the link "Prompt" and verify that the prompt dialog is displayed correctly. </p>
<a href="javascript: message()">Prompt</a> <br/>
</body>
</html>
Comment 1 Pat 2007-08-06 07:46:42 PDT
Created attachment 15849 [details]
test case reduction

test case reduction
Comment 3 Pat 2007-08-06 11:07:06 PDT
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.
Comment 4 Pat 2007-08-06 11:16:05 PDT
Created attachment 15852 [details]
3.2 patch for javascript in <a> href 

2nd patch for 3.2
Comment 5 Yongjun Zhang 2007-08-10 06:16:08 PDT
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.
Comment 6 Sornalatha Rathnasamy 2007-08-23 14:07:02 PDT
(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.
Comment 7 Bradley Morrison 2008-04-09 11:39:38 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 8 Joel Parks 2011-03-21 11:53:35 PDT
re-purposing InTSW keyword for use by QtWebkit team
Comment 9 Lucas Forschler 2019-02-06 09:02:30 PST
Mass moving XML DOM bugs to the "DOM" Component.