Bug 15049

Summary:

[S60] JKRL-769F2J: browser does not properly support the noscript tag correctly

Product:

WebKit

Reporter:

Wei Liu <wei.liu>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED INVALID

Severity:

Blocker

Keywords:

PlatformOnly

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Emulator

OS:

S60 3rd edition

Attachments:

Description	Flags
patch for both 3.1 and 3.2	zalan: review+
Additional changes to handling javascript disable.	zalan: review+

Wei Liu

Reported 2007-08-22 11:43:15 PDT

starting conditions: java scripting enabled (options settings -> general -> java/ecma script -> on) given the script: <?xml version="1.0"?> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML Basic 1.0//EN" "http://www.w3.org/TR/xhtml-basic/xhtml-basic10.dtd"> <html> <head> <title>Test of 'noscript' - VBcript</title> </head> <body> <p> Client should print either: VBscript supported! or VBScript NOT supported </p> <script type="text/vbscript">  </script> <noscript>VBScript NOT Supported!</noscript> </body> </html> browser should display "VBScript NOT Supported!" because it does not support the vbscript. in this case the content of the noscript element should be should be rendered because the browser is a script aware agent and it does not support the given script (the vbscript). for further reference check http://www.w3.org/TR/html4/interact/scripts.html section 18.3.1. also the rainbow browser (the s40 browser) properly supports the tag. please refer to the attached screenshots. this case block the oma copliancy certification. please fix as soon as possible.

Attachments
patch for both 3.1 and 3.2 (3.32 KB, patch) 2007-08-22 11:56 PDT, Wei Liu	zalan: review+	Details Formatted Diff Diff
Additional changes to handling javascript disable. (1.17 KB, patch) 2007-08-23 07:52 PDT, Wei Liu	zalan: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Wei Liu

Comment 1 2007-08-22 11:56:53 PDT

Created attachment 16075 [details] patch for both 3.1 and 3.2 Hi Zalan, This is an urgent showstopper bug, please review it at your earliest convenience. Sriram had also pre-reviewed it. If everything is fine, please check it in both 3.1m and 3.2 branch. Thanks a lot!

alan

Comment 2 2007-08-23 05:38:29 PDT

Comment on attachment 16075 [details] patch for both 3.1 and 3.2 r=sriram.yadavalli@nokia.com

alan

Comment 3 2007-08-23 05:42:44 PDT

25198->3.1m 25199->ccb

Wei Liu

Comment 4 2007-08-23 07:52:14 PDT

Created attachment 16096 [details] Additional changes to handling javascript disable.

alan

Comment 5 2007-08-23 12:21:41 PDT

25204->3.1m 25205->ccb

alan

Comment 6 2007-08-23 12:22:11 PDT

Comment on attachment 16096 [details] Additional changes to handling javascript disable. r=sriram.yadavalli@nokia.com

Sriram

Comment 7 2007-08-23 14:08:27 PDT

Please remove this patch

Bradley Morrison

Comment 8 2008-04-09 11:32:05 PDT

Bulk update of s60 bugs - tagging and closing.

Bradley Morrison

Comment 9 2008-04-09 11:39:56 PDT

Bulk closing of all s60 platform bugs. Sorry for the noise!

Joel Parks

Comment 10 2011-03-21 11:53:37 PDT

re-purposing InTSW keyword for use by QtWebkit team

Note You need to log in before you can comment on or make changes to this bug.