Clang style checker: turn on checks for nested classes

Clang style checker: turn on checks for nested classes

BUG=139346, 136863
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=151483

[hans, 2012-08-13 12:33:03]

This is blocked by https://chromiumcodereview.appspot.com/10824229/


Also, when I compile with this locally, I get "Unknown argument:
check-inner-classes" errors. It seems to work fine on the bots though :/

[hans, 2012-08-14 13:05:55]

All clean-ups are now in, so this should be good to go in.

The "Unknown argument: check-inner-classes" errors show up when I'm using goma,
but not otherwise. Something is probably just weird with my setup.

[Nico, 2012-08-14 15:04:17]

LGTM

I suppose you've run tryjobs with this before everything was done, so you know
the flag actually works :-)

After this lands, can you send a CL to turn this on by default in the plugin,
and to remove the flag from the plugin? (And once that's rolled in, we can
remove it from this file again.)

[hans, 2012-08-14 15:31:39]

On 2012/08/14 15:04:17, Nico wrote:
> LGTM
> 
> I suppose you've run tryjobs with this before everything was done, so you know
> the flag actually works :-)

Yup, see e.g. the red linux_clang bot above :)

> After this lands, can you send a CL to turn this on by default in the plugin,
> and to remove the flag from the plugin? (And once that's rolled in, we can
> remove it from this file again.)

OK, will do.