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
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
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: