1612828 - Remove `nsresult` from `Selection::FindInsertionPoint`

Status: RESOLVED FIXED

(Core :: DOM: Selection, enhancement)

Comment 1

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Attachment: Bug 1612828: part 1) Change signature of `Selection::SubtractRange`. r=smaug

Comment 2

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Attachment: Bug 1612828: part 2) Remove `nsresult` return type from`Selection::FindInsertionPoint`'s comparator`. r=smaug

Will allow to replace the hand-crafted binary search from the method.

Depends on D61531

Comment 3

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Attachment: Bug 1612828: part 3) Remove `nsresult` return type from `Selection::FindInsertionPoint`. r=smaug

Depends on D61532

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0c5d622f913a
part 1) Change signature of `Selection::SubtractRange`. r=smaug
https://hg.mozilla.org/integration/autoland/rev/364b0fe0560e
part 2) Remove `nsresult` return type from`Selection::FindInsertionPoint`'s comparator`. r=smaug
https://hg.mozilla.org/integration/autoland/rev/0d4b7b5b116b
part 3) Remove `nsresult` return type from `Selection::FindInsertionPoint`. r=smaug

Comment 5

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/0c5d622f913a
https://hg.mozilla.org/mozilla-central/rev/364b0fe0560e
https://hg.mozilla.org/mozilla-central/rev/0d4b7b5b116b