Moving test_api to api/test .

chrome/browser/extensions/api/test/test_api.cc

Index: chrome/browser/extensions/api/test/test_api.cc
diff --git a/chrome/browser/extensions/extension_test_api.cc b/chrome/browser/extensions/api/test/test_api.cc
similarity index 97%
rename from chrome/browser/extensions/extension_test_api.cc
rename to chrome/browser/extensions/api/test/test_api.cc
index a496678196d52ad9ec2e0f343a923dea322bfcb2..6c460ca8aee0a56cc7f5e5b1c11a8bfc84b60c17 100644
--- a/chrome/browser/extensions/extension_test_api.cc
+++ b/chrome/browser/extensions/api/test/test_api.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/extensions/extension_test_api.h"
+#include "chrome/browser/extensions/api/test/test_api.h"
 
 #include <string>
 
@@ -17,6 +17,8 @@
 #include "chrome/common/chrome_switches.h"
 #include "content/public/browser/notification_service.h"
 
+namespace extensions {

[Yoyo Zhou, 2012/07/10 19:00:11]

You could also put this at 34, so the anonymous namespace isn't in the
extensions namespace. I don't have a strong opinion on this; perhaps look at
some other files to see what the convention is.

[vabr (Chromium), 2012/07/11 10:01:25]

Done.
Both occur in the api/ subdirectory, with the one you suggest being more
frequent.

+
 namespace {
 
 // If you see this error in your test, you need to set the config state
@@ -139,3 +141,5 @@ bool ExtensionTestGetConfigFunction::RunImpl() {
   result_.reset(test_config_state->config_state()->DeepCopy());
   return true;
 }
+
+} // namespace extensions