Closed
Bug 1634663
Opened 5 years ago
Closed 5 years ago
Declare callers of `Selection::GetRangeAt` `const` when possible
Categories
(Core :: DOM: Selection, enhancement)
Core
DOM: Selection
Tracking
()
RESOLVED
FIXED
mozilla78
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: mbrodesser-Igalia, Assigned: mbrodesser-Igalia)
References
(Blocks 1 open bug)
Details
Attachments
(5 files)
Helps to determine where, outside of Selection, ranges are mutated.
Assignee | ||
Comment 1•5 years ago
|
||
Only where possible, of course.
Assignee | ||
Comment 2•5 years ago
|
||
Depends on D73444
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D73445
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D73446
Assignee | ||
Comment 5•5 years ago
|
||
Depends on D73447
Pushed by mbrodesser@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5d49d672bcaa
part 1) Declare `Selection::GetRangeAt`'s callers `const` around dom/ code. r=masayuki
https://hg.mozilla.org/integration/autoland/rev/3ba9cee0bfef
part 2) Declare callers of `GetRangeAt` around `EditorBase` `const`. r=masayuki
https://hg.mozilla.org/integration/autoland/rev/e25aeed66e9a
part 3) Declare callers of `Selection::GetRangeAt` in spellchecking code `const`. r=masayuki
https://hg.mozilla.org/integration/autoland/rev/e96619c151fc
part 4) Declare callers of `Selection::GetRangeAt` `const` in layout code. r=masayuki
https://hg.mozilla.org/integration/autoland/rev/d102cc6d3daf
part 5) Declare callers of `Selection::GetRangeAt` around toolkit/components/find `const`. r=masayuki
Updated•5 years ago
|
Severity: -- → N/A
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5d49d672bcaa
https://hg.mozilla.org/mozilla-central/rev/3ba9cee0bfef
https://hg.mozilla.org/mozilla-central/rev/e25aeed66e9a
https://hg.mozilla.org/mozilla-central/rev/e96619c151fc
https://hg.mozilla.org/mozilla-central/rev/d102cc6d3daf
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox78:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in
before you can comment on or make changes to this bug.
Description
•