FBTF: Forward declare and move constructors in chrome/common/extensions/.

FBTF: Forward declare and move constructors in chrome/common/extensions/.

TEST=none
BUG=none

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

[Elliot Glaysher, 2010-08-25 18:45:32]

AA: I saw your post in evanm's buzz on binary size, and since I'm devoting all
of Q3 to code cleanup, I did a quick pass through chrome/common/extensions/. (I
plan on doing a more through review latter; extensions is nowhere close to being
the worse offender in chrome.)

- Large chunks of the extension system get included through browser.h. I have
ideas about how to clean this up, but I'm going to have to work through a few
cleanup/forward declaration patches first.

- There's a lot of synthesized ctors/dtors/copy ctors/operator=. I've done a few
cleanups here and there, but things like user script need to have copy
constructors/operator=s added because those synthesized methods are huge.

- I've undone a template. Templates *suck* from a compile time and binary size.
This one probably didn't matter much, except it was blocked an #include removal.

(thestig: FYI for FBTF work)

[Lei Zhang, 2010-08-25 20:03:04]

Mostly LGTM, but I'll let aa have the final say.

http://codereview.chromium.org/3207002/diff/26/6021
File chrome/common/extensions/update_manifest.h (left):

http://codereview.chromium.org/3207002/diff/26/6021#oldcode12
chrome/common/extensions/update_manifest.h:12: #include "base/logging.h"
Can you exclude this bit? It's in my other CL already.

[Aaron Boodman, 2010-08-26 00:07:42]

This is great. Thanks for the fixes. LGTM