Closed
Bug 883712
Opened 11 years ago
Closed 4 years ago
"ASSERTION: Text leaf parent is not hypertext" with text node child of <tr>
Categories
(Core :: Disability Access APIs, defect, P3)
Core
Disability Access APIs
Tracking
()
People
(Reporter: jruderman, Unassigned)
References
Details
(Keywords: assertion, testcase, Whiteboard: a11y:crash-tree)
Attachments
(1 file)
714 bytes,
application/xhtml+xml
|
Details |
Similar to bug 852129, but with <tr> instead of <img>. 1. Enable accessibility. 2. Load the testcase. ###!!! ASSERTION: Text leaf parent is not hypertext!: 'Error', file accessible/src/base/TextUpdater.cpp, line 49
Comment 1•10 years ago
|
||
> Similar to bug 852129, but with <tr> instead of <img>.
yeah, same basic idea this time the accessible tree is
HTMLTableRowAccessible with text leaf accessible child.
the dom tree is just a <tr> with a text leaf child.
the frame tree is table row frame with child table cell frame with child block frame all having mContent be the <tr> and then a text frame.
I'm not really sure how the accessible tree should look here.
Comment 2•10 years ago
|
||
(In reply to Trevor Saunders (:tbsaunde) from comment #1) > > Similar to bug 852129, but with <tr> instead of <img>. > > the frame tree is table row frame with child table cell frame with child > block frame all having mContent be the <tr> and then a text frame. > > I'm not really sure how the accessible tree should look here. I'd try to rely on frame tree, i.e. expose what's supposedly on the screen if cell is optional for table then table row should probably implement text interface
Comment 3•7 years ago
|
||
Still reproduces on trunk.
Has Regression Range: --- → no
status-firefox56:
--- → wontfix
status-firefox57:
--- → wontfix
status-firefox58:
--- → fix-optional
status-firefox-esr52:
--- → wontfix
Comment 4•6 years ago
|
||
https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Move_fix-optionals
status-firefox59:
--- → ?
Comment 5•6 years ago
|
||
cannot reproduce on mac, Ryan, would you mind to give a try too?
Flags: needinfo?(ryanvm)
Comment 6•6 years ago
|
||
ok, I reprodcued it locally with moz_assert. The only problem I can see that text is not accessible with weird tables like the example's one. Putting p3 for it.
Flags: needinfo?(ryanvm)
Priority: -- → P3
Whiteboard: a11y:crash-tree
Pushed by surkov.alexander@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/79f04311c60c replace 'Text leaf parent is not hypertext' assertion to moz_assert
Updated•6 years ago
|
Keywords: leave-open
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/79f04311c60c
Comment 9•5 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months. :surkov, maybe it's time to close this bug?
Flags: needinfo?(surkov.alexander)
Comment 10•5 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:Jamie, maybe it's time to close this bug?
Flags: needinfo?(jteh)
Comment 11•5 years ago
|
||
The patch here just changes the call used to raise the assertion. It doesn't actually fix the bug, hence leave-open.
That said, I'm thinking this might be fixed by bug 1052866, since that added the HyperText interface to tables and table rows. This needs testing, though.
Flags: needinfo?(jteh)
Keywords: leave-open
Updated•5 years ago
|
Flags: needinfo?(surkov.alexander)
Updated•4 years ago
|
Whiteboard: a11y:crash-tree → a11y:crash-tree[mac2020_1]
Comment 12•4 years ago
|
||
I doubt this is a mac bug. Will double check later.
Comment 13•4 years ago
|
||
Yeah, it's not mac related.
OS: macOS → All
Hardware: x86_64 → All
Whiteboard: a11y:crash-tree[mac2020_1] → a11y:crash-tree
Comment 14•4 years ago
|
||
I can't reproduce this anymore. Going to close it assuming bug 1052866, fixed it.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•