Closed Bug 1731154 Opened 3 years ago Closed 3 years ago

Make AccAttributes entries movable and avoid copies.

Categories

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

defect

Tracking

()

RESOLVED FIXED
94 Branch
Tracking Status
firefox94 --- fixed

People

(Reporter: eeejay, Assigned: eeejay)

References

Details

Attachments

(1 file)

We currently need to use copyable arrays in AccAttributes. We should be avoiding copies and using references and moves whenever possible.

Changed the array type to nsTArray to avoid copies and get compile-time
errors if we ever try to do that. To set an array as a value, it must be
moved.

Assignee: nobody → eitan
Status: NEW → ASSIGNED
Blocks: 1730093

Changing severity to P1/S3 for triage

Severity: -- → S3
Priority: -- → P1
Pushed by eisaacson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/176469dcdeba
Add move semantics to AccAttributes and use value references to avoid copies. r=morgan

Backed out changeset 176469dcdeba (Bug 1731154) for causing bc failures on browser_attributed_text.js.
Backout link
Push with failures - bc4
Failure Log

Flags: needinfo?(eitan)
Pushed by eisaacson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/003ef3c8730c
Add move semantics to AccAttributes and use value references to avoid copies. r=morgan
Flags: needinfo?(eitan)
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: