Closed Bug 1521 Opened 26 years ago Closed 26 years ago

Background does not render.

Categories

(Core :: Layout, defect, P2)

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: christinehoff4, Assigned: rickg)

References

()

Details

1. Build: 11/20 - using release build, Viewer.exe 2. Platforms: Win 95, Win 98 3. Steps to reproduce: Open above URL in Viewer. 4. Expected results: Background should render. 5. Actual results: Background does not render.
Component: Rendering → Layout
Testing on 11/23 release build (Win 95), still a bug.
Assignee: michaelp → troy
Assignee: troy → rickg
This one's kind of interesting. Here's a small piece of HTML that demonstrates the problem: <HTML> <HEAD> <MAP></MAP> </HEAD> <BODY background="bg.gif"> Some text </BODY> When the content sink's AddAttributes() function is called the parser node for the BODY element has no attributes, i.e. PRInt32 ac = aNode.GetAttributeCount(); // XXX ac is '0'! It seems to be the <MAP> tag inside of the <HEAD> that causes the problem. If you remove it then the attribute handling is okay and the background renders
Assignee: rickg → michaelp
Assignee: michaelp → rickg
Oh brother... Rick, I assigned this but to you because it's a parser problem. See the nice little description down there Kipp explianed what's going on. You see the <MAP> inside of the <HEAD> and you auto-generate a BODY element. The auato-generated body element doesn't have the BACKGROUND attribute. Hence the rendering problem Kipp says you need to allow MAP to be inside of the HEAD
Summary: Background does not render. → ss:Background does not render.
Putting on ss: radar
Status: NEW → ASSIGNED
Summary: ss:Background does not render. → Background does not render.
*** Bug 1660 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Using 1/13 build on Win NT, Win 95 and Win 98: If I bring up the URL, there is nothing but a blue screen.
Resolution: FIXED → ---
Reopened bug re 1/13 comments
Setting all current Open/Normal to M4.
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
QA Contact: 4110
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Using 2_18 build, bug is fixed.
Verified fixed.
You need to log in before you can comment on or make changes to this bug.