Closed Bug 1140979 Opened 5 years ago Closed 5 years ago

Change SelectionHandler/TextSelection to use UUID vs. simple selectionID

Categories

(Firefox for Android :: Text Selection, defect)

Product:
Firefox for Android
Component:
Text Selection
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 39
Tracking Flags:
Tracking Status
firefox39 --- fixed

People

(Reporter: capella, Assigned: capella)

Details

Attachments

(1 file)

bug1140979.diff
4.72 KB, patch
wesj
: review+
Details | Diff | Splinter Review 
Looking at bug 988143, I'm planning to establish a TouchCaretHandler.js and a TouchSelectionHandler.js for the Gecko carets and while implementing in parallel with the existing scheme, we'll be using TextSelection/ActionBar as a shared service.

I need to have unique selectionID's generated after all, to avoid odd interractions, like we discussed in bug 1130258.
Attached patch bug1140979.diff — — Splinter Review
Attachment #8574548 - Flags: review?(wjohnston)
Attachment #8574548 - Flags: review?(wjohnston) → review+
https://hg.mozilla.org/mozilla-central/rev/1ee3f4cdae47
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
You need to log in before you can comment on or make changes to this bug.