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

NEW
Unassigned

Status

()

8 years ago
7 years ago

People

(Reporter: jruderman, Unassigned)

Tracking

({access, assertion, testcase})

Trunk
x86_64
Linux
access, assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
Created attachment 492935 [details]
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
(Reporter)

Comment 1

8 years ago
Created attachment 492936 [details]
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.

Updated

7 years ago
Keywords: access
You need to log in before you can comment on or make changes to this bug.