Move webCacheManager to //components.

chrome/browser/renderer_host/web_cache_manager_browsertest.cc

Index: chrome/browser/renderer_host/web_cache_manager_browsertest.cc
diff --git a/chrome/browser/renderer_host/web_cache_manager_browsertest.cc b/chrome/browser/renderer_host/web_cache_manager_browsertest.cc
index 7caee2f6ec2c893ec7e68d0636da5a64b5a9ee3d..1617728fece729063ecefaccd64a7208aedb6c82 100644
--- a/chrome/browser/renderer_host/web_cache_manager_browsertest.cc
+++ b/chrome/browser/renderer_host/web_cache_manager_browsertest.cc
@@ -5,12 +5,12 @@
 #include <string>
 
 #include "base/message_loop/message_loop.h"
-#include "chrome/browser/renderer_host/web_cache_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
+#include "components/web_cache/browser/web_cache_manager.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/result_codes.h"
@@ -18,6 +18,8 @@
 
 using content::WebContents;
 
+namespace web_cache {
+
 class WebCacheManagerBrowserTest : public InProcessBrowserTest {

[James Cook, 2014/09/05 16:53:18]

So what's the plan for these tests? It's weird that the test source files live
in src/chrome but the code they are testing lives in src/components.

I suggest doing something like we do for src/extensions unit tests. Move this
file into src/components along with the code under test, fix
src/components/web_cache/DEPS to allow src/chrome dependencies just test files,
then compile and run the tests as part of Chrome's browser_tests suite. Then
leave a comment and file a bug to track refactoring the tests so they can run in
components_browsertests.

[Xi Han, 2014/09/05 19:39:24]

I feel the same way. Thanks a lot for all the details, and I have moved it to
components:)

 };
 
@@ -67,3 +69,5 @@ IN_PROC_BROWSER_TEST_F(WebCacheManagerBrowserTest, CrashOnceOnly) {
   EXPECT_GE(WebCacheManager::GetInstance()->stats_.size(), 1U);
   EXPECT_LE(WebCacheManager::GetInstance()->stats_.size(), 2U);
 }
+
+}  // namespace web_cache