Move mock download test headers from content\test to content\public\test. This way we can enforce t…

Move mock download test headers from content\test to content\public\test. This way we can enforce that internal content headers don't leak to embedders.

BUG=98716
TBR=phajdan.jr 
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=140208

[Paweł Hajdan Jr., 2012-06-04 10:57:47]

LGTM

http://codereview.chromium.org/10501012/diff/1/content/public/test/mock_downl...
File content/public/test/mock_download_manager.h (right):

http://codereview.chromium.org/10501012/diff/1/content/public/test/mock_downl...
content/public/test/mock_download_manager.h:17: void PrintTo(const
DownloadRequestHandle& params, std::ostream* os);
This is ugly. Could you please add a TODO there (can be for me) to make it more
pretty? Thanks.

http://codereview.chromium.org/10501012/diff/1/content/test/mock_download_man...
File content/test/mock_download_manager.cc (right):

http://codereview.chromium.org/10501012/diff/1/content/test/mock_download_man...
content/test/mock_download_manager.cc:7: void PrintTo(const
DownloadRequestHandle& params, std::ostream* os) {
Won't this result in some weird double definition for PrintTo? If this is OK I'm
fine, I just don't remember how original PrintTo is defined, and cs.chromium.org
is not helpful to check this.