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

content/public/test/mock_download_manager.h

Index: content/public/test/mock_download_manager.h
===================================================================
--- content/public/test/mock_download_manager.h	(revision 140199)
+++ content/public/test/mock_download_manager.h	(working copy)
@@ -2,13 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CONTENT_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_
-#define CONTENT_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_
+#ifndef CONTENT_PUBLIC_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_
+#define CONTENT_PUBLIC_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_
 #pragma once
 
-#include "content/browser/download/download_request_handle.h"
-#include "content/common/content_export.h"
-#include "content/public/browser/download_item.h"
 #include "content/public/browser/download_manager.h"
 #include "content/public/browser/download_save_info.h"
 #include "content/public/browser/download_url_parameters.h"
@@ -16,6 +13,9 @@
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+// To avoid leaking download_request_handle.h to embedders.
+void PrintTo(const DownloadRequestHandle& params, std::ostream* os);

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

This is ugly. Could you please add a TODO there (can be for me) to make it more
pretty? Thanks.

+
 namespace content {
 
 class MockDownloadManager : public content::DownloadManager {
@@ -94,4 +94,4 @@
 
 }  // namespace content
 
-#endif  // CONTENT_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_
+#endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_MOCK_DOWNLOAD_MANAGER_H_