figure out why we can have a canvas frame with no children

NEW
Unassigned

Status

()

17 years ago
5 years ago

People

(Reporter: dbaron, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

This is a followup bug to bug 97226.  That bug was fixed with a null check that,
in theory, we shouldn't need, since a canvas frame should always (I think) have
children.  (Or is there a state of incremental loading where that might not be
true?)

Anyway, we should figure out whether the null check added for bug 97726 was the
right thing to do.  If it was, we should remove the comment that it's temporary.
 If it wasn't, we should do the right thing.
Bug 97226 is the right number (and bug 97726 is not).
Thanks David; I'll look at this once I get a build environment.

In the meantime, do we have any reliable ways of hitting the assertion? Does the
testcase in bug 97226 reliably hit it? From that bug:

------- Additional Comments From Samuel Sieb 2001-08-28 15:37 -------
I've reduced it down to this javascript file:
http://serve.m-ave.net/jscript/MerchandisingAvenue.js

------- Additional Comments From David Baron 2001-09-03 07:40 -------
The URLs reported in talkback report comments are:
http://www.kissasylum.com/
http://tvguide.com/magazine/covers
http://tvguide.com/
Status: NEW → ASSIGNED
Keywords: qawanted
Assignee: ian → nobody
Status: ASSIGNED → NEW
QA Contact: chrispetersen → core.layout

Comment 3

14 years ago
what is the status here?

The assertion and null check are still in nsCSSRending.cpp, but I was unable to
reproduce the assertion (A canvas with a background image had no child frame...)
using the URLs from bug 97226.

Updated

10 years ago
Depends on: 450319

Comment 4

5 years ago
No assertions reproducible with the links in bug 97226 on the 08/21 debug Nightly.
Keywords: qawanted
You need to log in before you can comment on or make changes to this bug.