Closed
Bug 1709715
Opened 3 years ago
Closed 3 years ago
Take advantage of CTAD for GetParticularProcessPriorityManager()
Categories
(Core :: DOM: Content Processes, task, P3)
Tracking
()
RESOLVED
FIXED
90 Branch
| Tracking | Status | |
|---|---|---|
| firefox90 | --- | fixed |
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
Attachments
(1 file)
This will let us use template argument inference in the places that call it. I'll wait to land this until bug 1618547 settles out.
|
||
Comment 1•3 years ago
|
||
We really need to just add type guides so CTAD works with already_AddRefed. Though I'm thinking at some point we should change our style guides to prefer returning RefPtr now that we have modern C++ features like move constructors and ref qualifiers.
|
Assignee | |
Comment 2•3 years ago
|
||
|
|
Assignee | |
Comment 3•3 years ago
|
||
Once bug 1710145 is fixed, this can just be about the part of my patch that drops the explicit template arguments.
Summary: Make GetParticularProcessPriorityManager() return a RefPtr → Take advantage of CTAD for GetParticularProcessPriorityManager()
|
||
Updated•3 years ago
|
Attachment #9220469 -
Attachment description: Bug 1709715 - Make GetParticularProcessPriorityManager() return a RefPtr. → Bug 1709715 - Take advantage of CTAD for GetParticularProcessPriorityManager().
Pushed by amccreight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/72ca306baa23 Take advantage of CTAD for GetParticularProcessPriorityManager(). r=kmag
|
||
Comment 5•3 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox90:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•