Closed Bug 27136 Opened 25 years ago Closed 25 years ago

All elements below the Style element are not displayed in page source.

Categories

(Core :: DOM: Editor, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: chrispetersen, Assigned: harishd)

Details

Attachments

(1 file)

Build: Apprunner Version: 2000020908 Platforms: All Expected Results: The page source should display all elements in HTML source. What I got: All elements below the Style element are not displayed in page source. Steps to reproduce: 1) Open attached file in Editor 2) Choose Save as from File menu and save file to local drive. 3) After saving file, select Page Source from View menu. 4) Notice that elements after Style element are not displayed in Page source.
Akkana -- can you walk through this and see what the problem is?
Assignee: beppe → akkana
Target Milestone: M15
I'll take a look, after beta.
Status: NEW → ASSIGNED
This only happens (for me, at least) if I modify the file before viewing source. What's happening is that when we save, the style element is being changed to: <html><head><title>HTML Functional -- ANCHOR - STYLE</title> <style type="text/css"> a { border-style: solid } </style><style></head> I.e. there's an extra open <style> tag inside the head, which is never closed. I assume this is confusing the source viewer, and it's certainly wrong. It's probably a bug in the output sink's handling of style tags.
Turns out this is a XIF DTD issue -- the parser is calling AddStartTag a second time for the style node. The XIF looks like this: <container isa="style"><attr name="type" value="text/css"/> </content>-style: solid } </container><!--style--> <css_stylesheet> <css_rule> <css_selector selectors="a"/> <css_declaration_list> <css_declaration property="border-top-style" value="solid"/> <css_declaration property="border-right-style" value="solid"/> <css_declaration property="border-bottom-style" value="solid"/> <css_declaration property="border-left-style" value="solid"/> </css_declaration_list> </css_rule> </css_stylesheet> and this gets translated to: <style type="text/css"> a { border-style: solid } </style><style> The stack trace for the bogus OpenContainer call is: #0 nsHTMLContentSinkStream::Write (this=0x8648048, aData=0x85dfb20 "style") at nsHTMLContentSinkStream.cpp:317 #1 0x4138bf44 in nsHTMLContentSinkStream::AddStartTag (this=0x8648048, aNode=@0xbfffcdcc) at nsHTMLContentSinkStream.cpp:753 #2 0x4138c52f in nsHTMLContentSinkStream::AddLeaf (this=0x8648048, aNode=@0xbfffcdcc) at nsHTMLContentSinkStream.cpp:913 #3 0x41395d36 in nsXIFDTD::EndCSSStyleSheet (this=0x860baa0, aNode=@0xbfffce74) at nsXIFDTD.cpp:1474 #4 0x413945b5 in nsXIFDTD::HandleEndToken (this=0x860baa0, aToken=0x832a878) at nsXIFDTD.cpp:683 #5 0x41393d31 in nsXIFDTD::HandleToken (this=0x860baa0, aToken=0x832a878, aParser=0x8647de0) at nsXIFDTD.cpp:464 Harish and I will be looking at some XIF DTD issues in M15.
Assignee: akkana → harishd
Status: ASSIGNED → NEW
Moving to M17
Target Milestone: M15 → M17
I've rewritten XIF DTD...and in the process this bug got fixed ( in my local tree ). Will check my changes in after a through testing.. ETA: Monday (04/17/00)
Okay, should be fixed now. Marking FIXED.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Verifying fixed in the Oct 30th build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: