RESOLVED WONTFIX Bug 83645
[chromium] [meta] We should use OVERRIDE annotations where appropriate in chromium compositor code 
https://bugs.webkit.org/show_bug.cgi?id=83645
Summary [chromium] [meta] We should use OVERRIDE annotations where appropriate in chr...
James Robinson
Reported 2012-04-10 17:52:36 PDT
Annotating implementations of virtual functions that we intend to be overrides with OVERRIDE could help us cut down on bugs and make refactors easier. Currently OVERRIDE is supported on the following chromium configurations: 1.) Windows - MSVS2010, on by default. Enabled on the bots. 2.) Mac - clang, on by default. Enabled on the bots. 3.) Linux - default compiler (gcc 4.4.3 that comes with lucid) ignores OVERRIDE, clang (very easy to set up, we have non-default bots that run it) supports it. Not enabled on most bots - in particular cr-linux EWS and commit queue don't use clang currently. Many of use who work on the compositor guts code on linux, but we could get local checking for OVERRIDE by switching to clang. Regardless, we'd still get coverage from many bots. On balance I think this would be a win. I was bit while doing https://bugs.webkit.org/show_bug.cgi?id=83530 when I forgot to update a unit test implementation that was trying to override an interface I had refactored and I can think of a few instances in the past where this has caused trouble for developers and caused a few real shipping bugs.
Attachments
Add attachment proposed patch, testcase, etc.
Note You need to log in before you can comment on or make changes to this bug.