Bug 83645 - [chromium] [meta] We should use OVERRIDE annotations where appropriate in chromium compositor code
Summary: [chromium] [meta] We should use OVERRIDE annotations where appropriate in chr...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 83646 83650 83653 83658 87073 87074
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-10 17:52 PDT by James Robinson
Modified: 2013-04-15 07:01 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description James Robinson 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.