Closed Bug 1823979 Opened 2 years ago Closed 2 years ago

Assertion failure: aPoint.mAcc->IsDoc() || content->HasFlag(NODE_IS_EDITABLE)

Categories

(Core :: Disability Access APIs, defect, P3)

Product:
Core
Component:
Disability Access APIs
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
113 Branch
Tracking Flags:
Tracking Status
firefox113 --- fixed

People

(Reporter: morgan, Assigned: eeejay)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1823979 - Remove strict assertion in text selection. r?morgan!
(deleted), text/x-phabricator-request
Details

STR:

  1. In a debug build, activate VoiceOver
  2. Load data:text/html,<input type="datetime-local">
  3. Navigate to the calendar button and activate it

Expected:
The calendar opens

Actual:
We hit this assertion

Eitan, could you take a look at this? You added this assertion with the selection stuff in bug 1811092.

Flags: needinfo?(eitan)
Blocks: a11y-ctw-text
Severity: -- → S3

I'm going to remove the assertion. I was trying to anticipate all the conditions we are in a non-text leaf. But, these kinds of esoteric markups where a button could be in an input show that it isn't worth chasing all the edge cases for a "correct" assertion statement.

Flags: needinfo?(eitan)

I was trying to anticipate all the conditions that would bring us
to a non-text leaf text point in a text selection. But it is hard to
account for other "correct" cases so this assertion should probably just
go away.

Assignee: nobody → eitan
Status: NEW → ASSIGNED
Pushed by eisaacson@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1fb37b9b29da Remove strict assertion in text selection. r=morgan

https://hg.mozilla.org/mozilla-central/rev/1fb37b9b29da

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox113: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: