FBTF: Move virtual methods to implementation files.

FBTF: Move virtual methods to implementation files.

Remove logging.h and other headers where possible.

BUG=none
TEST=none

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

[Elliot Glaysher, 2010-09-23 21:39:02]

Most of these are nits. Some of them actually *do* matter. Obviously the
filters.h refactoring is huge, but the IPC::SyncChannel modifications are
measurable, which surprised me. Same with other virtual methods that return
things or call methods.

[Lei Zhang, 2010-09-23 22:05:30]

LGTM with nits below:

http://codereview.chromium.org/3461019/diff/5001/6001
File app/table_model.cc (right):

http://codereview.chromium.org/3461019/diff/5001/6001#newcode89
app/table_model.cc:89: NOTREACHED();
can you #include logging.h and not implicitly depend on another header to
#include it?

http://codereview.chromium.org/3461019/diff/5001/6031
File chrome/browser/extensions/extension_infobar_delegate.cc (right):

http://codereview.chromium.org/3461019/diff/5001/6031#newcode62
chrome/browser/extensions/extension_infobar_delegate.cc:62:
AsExtensionInfoBarDelegate() {
style-wise, is this the better way to wrap really long method names or is it
better to write:

ExtensionInfoBarDelegate*
ExtensionInfoBarDelegate::AsExtensionInfoBarDelegate()

?