1355 - Background images in tables don't tile correctly

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect, P3)

Description

[Angus Davis]

Here's a simple test case. The image should tile across the whole table, not
each individual cell:

<table style="background-image:
url(http://home.netscape.com/images/ns_netcenter_bar.gif);">
<tr>
<td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td></tr>
<tr>
<td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td></tr>
<tr>
<td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td></tr>
<tr>
<td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td><td>asdas</td></tr>
</table>

Comment 1

[buster]

(when the following bug is fixed, reassign this to karnaze)
the css parser crashes calling netlib with an unexpected null arg.
the stack is:
NS_MakeAbsoluteURL(nsIURL * 0x00000000, const nsString & {...}, const nsString &
{...}, nsString & {...}) line 1009 + 7 bytes
CSSParserImpl::ParseURL(int & 0, nsCSSValue & {...}) line 1907 + 28 bytes
CSSParserImpl::ParseVariant(int & 0, nsCSSValue & {...}, int 393232, int *
0x00000000, int & 0) line 1743 + 19 bytes
CSSParserImpl::ParseVariant(int & 0, nsICSSDeclaration * 0x01278940, int 5, int
393232, int * 0x00000000, int & 0) line 1647 + 28 bytes
CSSParserImpl::ParseProperty(int & 0, nsICSSDeclaration * 0x01278940, int 5)
line 2038 + 31 bytes
CSSParserImpl::ParseProperty(int & 0, nsICSSDeclaration * 0x01278940, int 5, int
& 3) line 2011
CSSParserImpl::ParseDeclaration(int & 0, nsICSSDeclaration * 0x01278940, int 0,
int & 3) line 1397 + 24 bytes
CSSParserImpl::ParseDeclarationBlock(int & 0, int 0) line 1256 + 24 bytes
CSSParserImpl::ParseDeclarations(CSSParserImpl * const 0x012786f0, const
nsString & {...}, nsIURL * 0x00000000, nsIStyleRule * & 0x0012dc08) line 383 +
14 bytes
nsGenericHTMLElement::SetAttribute(int 3, nsIAtom * 0x01200480, const nsString &
{...}, int 0) line 519 + 22 bytes
nsHTMLTableElement::SetAttribute(nsHTMLTableElement * const 0x0127857c, int 3,
nsIAtom * 0x01200480, const nsString & {...}, int 0) line 112 + 30 bytes
AddAttributes(const nsIParserNode & {...}, nsIHTMLContent * 0x0127857c,
nsIScriptContextOwner * 0x011ee4ac) line 474
SinkContext::OpenContainer(const nsIParserNode & {...}) line 950 + 17 bytes
HTMLContentSink::OpenContainer(HTMLContentSink * const 0x01228c90, const
nsIParserNode & {...}) line 1769
CNavDTD::OpenContainer(const nsIParserNode & {...}, int 1) line 2789 + 22 bytes
CNavDTD::HandleDefaultStartToken(CToken * 0x0126f670, nsHTMLTag eHTMLTag_table,
nsIParserNode & {...}) line 845 + 14 bytes
CNavDTD::HandleStartToken(CToken * 0x0126f670) line 943 + 23 bytes
NavDispatchTokenHandler(CToken * 0x0126f670, nsIDTD * 0x0126ec30) line 446 + 12
bytes
CTokenHandler::operator()(CToken * 0x0126f670, nsIDTD * 0x0126ec30) line 80 + 14
bytes
CNavDTD::HandleToken(CNavDTD * const 0x0126ec30, CToken * 0x0126f670, nsIParser
* 0x01228be0) line 697 + 18 bytes
nsParser::BuildModel() line 765 + 20 bytes
nsParser::ResumeParse() line 723 + 11 bytes

Comment 2

[Peter Linss]

*** Bug 2269 has been marked as a duplicate of this bug. ***

Comment 3

[Peter Linss]

*** Bug 2375 has been marked as a duplicate of this bug. ***

Comment 4

[Peter Linss]

Crasher fixed.

Comment 5

[leger]

Setting all current Open Critical and Major to M3

Comment 6

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 7

[Christine Hoffman]

Reassigning qa content to [e-mail]

Comment 8

[karnaze (gone)]

Changed the summary and downgraded the priority and severity. This page no
longer crashes on win32 debug.

Comment 9

[karnaze (gone)]

This works the way Nav4.5 does, but not the way the spec says. By removing the
background attributes from the table related elements in ua.css, the url can be
made to tile correctly. Thus it appears we may need to have 2 different ua.css
files, one for Standard mode and one for Nav Quirks mode. They may even cascade.
I'm reassigning this to Peter.

Comment 10

[Peter Linss]

This is much improved (test case works as expected). There are still some issues
that I'd like to address later, so I'm leaving this open but changing the
target...

Comment 11

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

This is related to bug 4510.

Comment 12

[karnaze (gone)]

Fixed with latest checkin. Standard mode needs to be set (Nav Quirks is the
default mode).

Comment 13

[Jan Carpenter]

Verified fixt in standard mode.
build: 06-15-16-M7

Comment 14

[Gunnar Liljas]

The same problem occurs when using the BACKGROUND property of a TABLE tag.

Comment 15

[Jason Summers]

(concerning background image tiling) What is the justification for reproducing 
this peculiar behavior of NS4, even in Quirks mode? It's very annoying, and a 
frequent complaint in HTML newsgroups. There can't be many existing web pages 
that depend on it, can there? Anyway, isn't Quirks mode supposed to be mainly 
for bugs that occur in MSIE as well as NS4?

Comment 16

[Greg K.]

Attachment: The Bugzilla Ant with example IPTC metadata added.

Comment 17

[Greg K.]

Comment on attachment 54518 [details]
The Bugzilla Ant with example IPTC metadata added.

Was intended for bug 106068.