Closed Bug 1235513 Opened 9 years ago Closed 8 years ago

Remove SelectionHandler.js and optional use of original Java selection carets

Categories

(Firefox for Android Graveyard :: Text Selection, defect)

defect
Not set
normal

Tracking

(firefox52 fixed)

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: capella, Assigned: kats)

References

Details

Attachments

(1 file)

While m-c builds now default to the new AccessibleCaret selection handlers that core provides, mobile users can currently opt to revert to the original Java carets by setting |layout.accessiblecaret.enabled| to false.

We should remove that as an option, and it's attendant code, SelectionHandler.js.
Blocks: gecko-carets
Blocks: text-selection-mvp
No longer blocks: gecko-carets
More code deletion, sign me up!
Assignee: nobody → bugmail
Comment on attachment 8798388 [details]
Bug 1235513 - Remove the old Fennec SelectionHandler now that it has been replaced by the AccessibleCaret code in platform.

https://reviewboard.mozilla.org/r/83904/#review82450
Attachment #8798388 - Flags: review?(snorp) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00c81c0c177b
Remove the old Fennec SelectionHandler now that it has been replaced by the AccessibleCaret code in platform. r=snorp
Depends on: 1308371
https://hg.mozilla.org/mozilla-central/rev/00c81c0c177b
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Depends on: 1339797
No longer depends on: 1339797
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: