[Webkit-unassigned] [Bug 56393] Without checking existence of the renderer of the element, tries to access the enclosing layer.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 14 13:36:49 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=56393





--- Comment #28 from Alexis Menard <alexis.menard at openbossa.org>  2011-06-14 13:36:49 PST ---
(In reply to comment #26)
> (In reply to comment #24)
> > > enclosingLayer().
> > The first idea was better than the current one.
> > One thing bother me. If the test is good enough you don't need the ASSERT, it should just crash properly therefore it will be catch at a LayoutTest failure. A crashing test is good enough to be treated as a regression/failure.
> 
> That's the problem.  The test without the ASSERT is not good enough
> to crash on the Unix platforms.  The Unix platforms allow a null
> "this" pointer and Symbian does not.  I can't think of a way to 
> make a better test.

Either ASSERT earlier or make a specific layout test only executed on Symbian (look in LayoutTests/platform/qt, we have different sets of Skipped tests, e.g. one with Qt 4.8).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list