Cleanup forward declared classes that are not needed.

Cleanup forward declared classes that are not needed in chrome/renderer, chrome/test/, and chrome/views. Also fix some lint issues while I'm at it.

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

[Evan Stade, 2009-02-05 23:38:23]

DISALLOW_COPY_AND_ASSIGN is preferred to DISALLOW_EVIL_CONSTRUCTORS (as long as
you're touching that stuff might as well change it)

And I dunno if it's a rietveld bug or what, but it looks like you're deleting
some file-ending newlines?

[Lei Zhang, 2009-02-05 23:51:42]

On 2009/02/05 23:38:23, estade wrote:
> DISALLOW_COPY_AND_ASSIGN is preferred to DISALLOW_EVIL_CONSTRUCTORS (as long
as
> you're touching that stuff might as well change it)

ok, s/DISALLOW_EVIL_CONSTRUCTORS/DISALLOW_COPY_AND_ASSIGN/

> And I dunno if it's a rietveld bug or what, but it looks like you're deleting
> some file-ending newlines?

Probably a rietveld bug. Looks fine on my box.

[Evan Stade, 2009-02-05 23:59:36]

I actually don't think it's a rietveld bug. Look at the last lines of the raw
patch:

 #endif  // CHROME_RENDERER_RENDER_PROCESS_H__
-

LGTM with EOF newlines fixed