Move RuntimeData and related permissions out of Extension class

chrome/common/extensions/manifest_tests/extension_manifests_chromepermission_unittest.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 "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/manifest.h"
 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
+#include "chrome/common/extensions/permissions/permissions_data.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace errors = extension_manifest_errors;
 
+namespace extensions {
+
 TEST_F(ExtensionManifestTest, ChromeURLPermissionInvalid) {
   LoadAndExpectError("permission_chrome_url_invalid.json",
                      errors::kInvalidPermissionScheme);
 }
 
 TEST_F(ExtensionManifestTest, ChromeURLPermissionAllowedWithFlag) {
   CommandLine::ForCurrentProcess()->AppendSwitch(
       switches::kExtensionsOnChromeURLs);
   std::string error;
-  scoped_refptr<extensions::Extension> extension =
+  scoped_refptr<Extension> extension =
     LoadAndExpectSuccess("permission_chrome_url_invalid.json");
   EXPECT_EQ("", error);
   const GURL newtab_url("chrome://newtab/");
-  EXPECT_TRUE(extension->CanExecuteScriptOnPage(newtab_url,
-                                                newtab_url,
-                                                0,
-                                                NULL,
-                                                &error)) << error;
+  EXPECT_TRUE(PermissionsData::CanExecuteScriptOnPage(
+      extension, newtab_url, newtab_url, 0, NULL, &error)) << error;
 }
 
 TEST_F(ExtensionManifestTest, ChromeResourcesPermissionValidOnlyForComponents) {
   LoadAndExpectError("permission_chrome_resources_url.json",
                      errors::kInvalidPermissionScheme);
   std::string error;
   LoadExtension(Manifest("permission_chrome_resources_url.json"),
-                &error, extensions::Manifest::COMPONENT,
-                extensions::Extension::NO_FLAGS);
+                &error,
+                extensions::Manifest::COMPONENT,

[Yoyo Zhou, 2013/05/11 00:57:04]

nit: in extensions namespace

[Devlin, 2013/05/13 22:44:23]

But there's also a ExtensionManifestTest::Manifest class
(https://code.google.com/p/chromium/codesearch#chromium/src/chrome/common/exte...),
and the compiler will prefer that over extensions::Manifest - so we need to
specify which Manifest class we want.

+                Extension::NO_FLAGS);
   EXPECT_EQ("", error);
 }
+
+}  // namespace extensions