Open Bug 614501 Opened 14 years ago Updated 2 years ago

"ASSERTION: should only care when we have an outside bullet" with accessibility enabled

Categories

(Core :: Layout, defect)

x86_64
Linux
defect

Tracking

()

People

(Reporter: jruderman, Unassigned)

References

Details

(Keywords: access, assertion, testcase)

Attachments

(2 files)

Attached file testcase (deleted) —
1. Load the testcase 2. In the JS console, paste Components.classes["@mozilla.org/accessibilityService;1"] .getService(Components.interfaces.nsIAccessibleRetrieval); 3. Focus the testcase(?) ###!!! ASSERTION: should only care when we have an outside bullet: 'mContent->GetPrimaryFrame()->GetStyleDisplay()->mDisplay == NS_STYLE_DISPLAY_LIST_ITEM && HaveOutsideBullet()', file /builds/slave/mozilla-central-linux64-debug/build/layout/generic/nsBlockFrame.cpp, line 6542
Attached file stack trace (deleted) —
I think BulletIsEmptyExternal() should check for HaveOutsideBullet()...
Blocks: 557795
(In reply to comment #2) > I think BulletIsEmptyExternal() should check for HaveOutsideBullet()... What is HaveOutsideBullet for?
Bullets can be inside or outside. See http://www.w3.org/TR/CSS21/generate.html#propdef-list-style-position Does a11y need to create a bullet accessible in both cases?
(In reply to comment #4) > Does a11y need to create a bullet accessible in both cases? I think so.
Then it sounds like we need a way to make BulletIsEmpty() work for inside bullets, right?
(In reply to comment #6) > Then it sounds like we need a way to make BulletIsEmpty() work for inside > bullets, right? Yes. Btw, what about GetBulletText(), does it work for inside and outside bullets well?
Yes. For that matter, BulletIsEmpty seems like it should work too, except for the assert bit.
Keywords: access
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: