Last Comment Bug 45555 - SCRIPT in BODY doesn't work when strict dtd is used
: SCRIPT in BODY doesn't work when strict dtd is used
Status: VERIFIED FIXED
[nsbeta3+][Fix in hand]
:
Product: Core
Classification: Components
Component: HTML: Parser (show other bugs)
: Trunk
: x86 Windows 2000
: P3 normal (vote)
: ---
Assigned To: harishd
: Jan Carpenter
: Andrew Overholt [:overholt]
Mentors:
http://homepage1.nifty.com/emk/moz/sc...
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2000-07-15 00:33 PDT by Masatoshi Kimura [:emk]
Modified: 2000-10-16 14:32 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Testcase (249 bytes, text/html)
2000-07-15 00:34 PDT, Masatoshi Kimura [:emk]
no flags Details
Proposed patch. (1005 bytes, patch)
2000-07-26 09:30 PDT, harishd
no flags Details | Diff | Splinter Review

Description Masatoshi Kimura [:emk] 2000-07-15 00:33:29 PDT
Steps to reproduce:
1. Launch Mozilla.
2. Navigate to the following testcase.

Actual result:
An alert doesn't popup.

Expected result:
An alert should popup.

Occurs on:
2000071420 nightly on Windows 2000.

Strict dtd allows SCRIPT in BODY.
( http://www.w3.org/TR/html4/strict.dtd )
|<!ELEMENT BODY O O (%block;|SCRIPT)+ +(INS|DEL) -- document body -->
Comment 1 Masatoshi Kimura [:emk] 2000-07-15 00:34:24 PDT
Created attachment 11443 [details]
Testcase
Comment 2 harishd 2000-07-17 15:28:09 PDT
BODY should be able to contain SCRIPT ( per spec. ). There is not excuse for not 
following the spec. We have to fix this problem for beta3. IT IS MUST.
Comment 3 Masatoshi Kimura [:emk] 2000-07-20 14:15:50 PDT
If script is external, it is not ignored, but incorrect.
Here is a testcase.

http://homepage1.nifty.com/emk/moz/script.html -
"text2" is drawn, then "text1" is drawn and "text2" disappears.

http://homepage1.nifty.com/emk/moz/script-quirks.html -
"text1" and "text2" is drawn.
Comment 4 harishd 2000-07-26 09:30:56 PDT
Created attachment 11944 [details] [diff] [review]
Proposed patch.
Comment 5 harishd 2000-07-26 09:39:41 PDT
External JS problem will get resolved if I combine the above patch with the
patch in bug 44285.

Cause: Block error message ( sent by the script engine on loading an external JS
) got lost in COtherElements.h.
Comment 6 Nisheeth Ranjan 2000-08-04 10:51:39 PDT
Trivial fix for a relatively serious problem.  Marking nsbeta3+.
Comment 7 harishd 2000-08-09 14:43:42 PDT
Fix is in. Marking FIXED.
Comment 8 Masatoshi Kimura [:emk] 2000-08-21 11:02:06 PDT
Verified fixed using 2000082104 nightly build on Windows 2000.

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