Clean up 'virtual' annotations in print-related frame classes
Categories
(Core :: Printing: Output, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox80 | --- | fixed |
People
(Reporter: dholbert, Assigned: dholbert)
Details
Attachments
(2 files)
The nsPage*Frame
classes have some unnecessary virtual annotations that we should clean up.
Filing this bug on doing that.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
These classes are both 'final', so it's pointless for them to declare their own
(non-overriding) virtual functions. They can't have any subclasses, so it would
be impossible for there to be polymorphism in these virtual funtions'
implementation. So, let's de-virtualize them.
(I'm guessing these date from the early Mozilla days when methods needed to be
virtual in order to be called from other components, or something like that.)
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
Per Mozilla coding style:
"Method declarations must use, at most, one of the following keywords:
virtual, override, or final"
https://firefox-source-docs.mozilla.org/code-quality/coding-style/coding_style_cpp.html#c-c
Depends on D83234
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/40b817b70019
https://hg.mozilla.org/mozilla-central/rev/4328a6c14818
Description
•