[Webkit-unassigned] [Bug 83645] New: [chromium] [meta] We should use OVERRIDE annotations where appropriate in chromium compositor code

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=83645

           Summary: [chromium] [meta] We should use OVERRIDE annotations
                    where appropriate in chromium compositor code
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jamesr at chromium.org
                CC: enne at google.com, cc-bugs at google.com


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.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.