Moving test_api to api/test .

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

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // 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>
 
 #include "base/command_line.h"
 #include "base/memory/singleton.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_function_dispatcher.h"
 #include "chrome/browser/extensions/extensions_quota_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/common/chrome_notification_types.h"
 #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
 // to be returned by chrome.test.getConfig().  Do this by calling
 // ExtensionTestGetConfigFunction::set_test_config_state(Value* state)
 // in test set up.
 const char kNoTestConfigDataError[] = "Test configuration was not set.";
 
 const char kNotTestProcessError[] =
     "The chrome.test namespace is only available in tests.";
@@ skipping 102 matching lines @@
   TestConfigState* test_config_state = TestConfigState::GetInstance();
 
   if (!test_config_state->config_state()) {
     error_ = kNoTestConfigDataError;
     return false;
   }
 
   result_.reset(test_config_state->config_state()->DeepCopy());
   return true;
 }
+
+} // namespace extensions