Fix style in remote inspector classes Make some things `const` and use `final` instead of `override` when possible.
Created attachment 328726 [details] [PATCH] Proposed Fix
Comment on attachment 328726 [details] [PATCH] Proposed Fix Can you put some of the final methods as private instead of public?
We probably can, but that rarely makes sense to me, especially if it was public in a parent class. I know I've seen this pattern elsewhere though, maybe someone can explain the advantage of that to me.
We usually try to restrict the public methods to the minimum. With this pattern, if B derives from A and C is the method being overridden, C can only be called in the context of code using A references/pointers. There may be valid cases for which C can be called from B classes in which case B::C can be made public, but this does not seem to happen a lot.
Comment on attachment 328726 [details] [PATCH] Proposed Fix Clearing flags on attachment: 328726 Committed r225654: <https://trac.webkit.org/changeset/225654>
All reviewed patches have been landed. Closing bug.