Closed Bug 1826848 Opened 2 years ago Closed 1 years ago

[CtW] Assert cache is disabled before calling sync IPDL calls

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Platform:
Desktop
Unspecified
Type:
defect

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: eeejay, Assigned: eeejay)

References

(Blocks 1 open bug)

Details

Attachments

(1 obsolete file)

There are several methods in RemoteAccessible that should never be called when caching is enabled. We need to assert the state of the pref in those functions.

A list of such methods:

  • NativeState
  • Relations
  • SelectionBoundsAt
  • TableOfACell
  • GetColRowExtents
  • GetPosition
  • ColExtent
  • TableCaption
  • TableSummary
  • TableColumnCount
  • TableRowCount
  • TableCellAt
  • TableCellIndexAt
  • TableColumnIndexAt
  • TableRowIndexAt
  • TableRowAndColumnIndicesAt
  • TableColumnExtentAt
  • TableRowExtentAt
  • TableColumnDescription
  • TableRowDescription
  • TableColumnSelected
  • TableRowSelected
  • TableCellSelected
  • TableSelectedCellCount
  • TableSelectedColumnCount
  • TableSelectedRowCount
  • TableSelectedCells
  • TableSelectedCellIndices
  • TableSelectedColumnIndices
  • TableSelectedRowIndices
  • TableSelectColumn
  • TableSelectRow
  • TableUnselectColumn
  • TableUnselectRow
  • AtkTableColumnHeader
  • AtkTableRowHeader
  • AtkKeyBinding
Severity: -- → S4

SelectionBoundsAt too

Extents as well.

Assignee: nobody → eitan
Status: NEW → ASSIGNED

Assertions are also needed for ColIdx, RowIdx, RowExtent, ColHeaderCells, RowHeaderCells, IsCellSelected.

Comment on attachment 9328050 [details]
Bug 1826848 - Fix ATK extents to used cached bounds. r?morgan

Revision D175187 was moved to bug 1826870. Setting attachment 9328050 [details] to obsolete.

Attachment #9328050 - Attachment is obsolete: true

Superseded by bug 1821957, where I remove all these methods completely. :)

Status: ASSIGNED → RESOLVED
Closed: 1 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: