Added new gmock-based MockFilterHost and deprecated the old one.

Added new gmock-based MockFilterHost and deprecated the old one.

Updated FileDataSource tests to use the new MockFilterHost as a proof of concept, also because my pipeline refactoring completely broke the test.

BUG=16008
TEST=FileDataSourceTest.* should pass

[scherkus (not reviewing), 2009-07-08 23:03:47]

More gmock goodness

[awong, 2009-07-08 23:06:53]

LGTM

http://codereview.chromium.org/149350/diff/1001/31
File media/filters/file_data_source.h (right):

http://codereview.chromium.org/149350/diff/1001/31#newcode41
Line 41: // of my tests!!!
Ha ha ha...come over to the camp of pattern hatters.  It's simpler over here. :)

[Alpha Left Google, 2009-07-08 23:20:46]

LGTM except for the use of namespace in file_data_source_unittest.cc

http://codereview.chromium.org/149350/diff/1001/32
File media/filters/file_data_source_unittest.cc (right):

http://codereview.chromium.org/149350/diff/1001/32#newcode18
Line 18: namespace {
should you remove this? It feels like you are nesting the namespace.

[awong, 2009-07-08 23:23:28]

http://codereview.chromium.org/149350/diff/1001/32
File media/filters/file_data_source_unittest.cc (right):

http://codereview.chromium.org/149350/diff/1001/32#newcode18
Line 18: namespace {
On 2009/07/08 23:20:47, Alpha wrote:
> should you remove this? It feels like you are nesting the namespace.
> 

There's no problem with nesting an anonymous namespace I don't think.  Howeever,
it is truet hat you don't seem to be using anything inside the media namespace.

[scherkus (not reviewing), 2009-07-09 00:03:29]

removed anon namespace