Add OVERRIDE to jingle/.

Add OVERRIDE to jingle/.

BUG=104314
TEST=no change

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110546

[Avi (use Gerrit), 2011-11-16 22:53:38]

Need owner approval. Just adding OVERRIDE everywhere.

[akalin, 2011-11-17 02:57:56]

On 2011/11/16 22:53:38, Avi wrote:
> Need owner approval. Just adding OVERRIDE everywhere.

Don't you also need to #include "base/compiler_specific.h" from these files?

[Avi (use Gerrit), 2011-11-17 06:15:29]

Technically speaking, perhaps. In reality I'm using a clang plugin, and
since a few key headers already have that include, it trickles down until I
have to use just a few of them here and there.

[akalin, 2011-11-17 11:25:45]

On 2011/11/17 06:15:29, Avi wrote:
> Technically speaking, perhaps. In reality I'm using a clang plugin, and
> since a few key headers already have that include, it trickles down until I
> have to use just a few of them here and there.

Well, by "need" i meant according to the style guide (include what you use), and
not that it'll break if you don't do it.

Can you modify your clang plugin to add the include if it's not already there? 
I'm sure a perl script could do it.

[Avi (use Gerrit), 2011-11-17 15:52:25]

The Clang plugin we have doesn't alter the code.

If you want I can add it to these files. But it's used everywhere and the header
is never included.

[akalin, 2011-11-17 18:12:18]

On 2011/11/17 15:52:25, Avi wrote:
> The Clang plugin we have doesn't alter the code.
> 
> If you want I can add it to these files. But it's used everywhere and the
header
> is never included.

Yeah, I'd like you to add it to these files.  Thanks!

[Avi (use Gerrit), 2011-11-17 19:55:30]

ptal

[akalin, 2011-11-17 20:03:55]

LGTM, thanks!