Remove SandboxedPages and SandboxedCSP from Extension Class

chrome/common/extensions/sandboxed_handler.h

Index: chrome/common/extensions/sandboxed_handler.h
diff --git a/chrome/common/extensions/sandboxed_handler.h b/chrome/common/extensions/sandboxed_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..b06e2d9dfe398b88e1481ba7bcfcdee9c501d0b7
--- /dev/null
+++ b/chrome/common/extensions/sandboxed_handler.h
@@ -0,0 +1,56 @@
+// Copyright (c) 2013 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.
+
+#ifndef CHROME_COMMON_EXTENSIONS_SANDBOXED_HANDLER_H_
+#define CHROME_COMMON_EXTENSIONS_SANDBOXED_HANDLER_H_
+
+#include <string>
+
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/manifest_handler.h"
+#include "extensions/common/url_pattern_set.h"
+
+namespace extensions {
+
+struct SandboxedInfo : public Extension::ManifestData {

[Yoyo Zhou, 2013/03/20 22:53:53]

I'd prefer a name like SandboxedPageInfo.

Since most users of this are going to refer to the Info rather than the Handler,
how about renaming this file to sandboxed_page_info?

[Devlin, 2013/03/23 22:26:25]

Done.

+ public:
+  SandboxedInfo();
+  virtual ~SandboxedInfo();
+
+  // Returns the extension's Content Security Policy for the sandboxed pages.
+  static const std::string& GetContentSecurityPolicy(
+      const Extension* extension);
+
+  // Returns the extension's sandboxed pages.
+  static const URLPatternSet& GetPages(const Extension* extension);
+
+  // Returns true if the specified page is sandboxed (served in a unique
+  // origin).

[Yoyo Zhou, 2013/03/20 22:53:53]

either "...with a different CSP" or delete the parenthetical

[Devlin, 2013/03/23 22:26:25]

Done.

+  static bool IsSandboxedPage(const Extension* extension,
+                              const std::string& relative_path);
+
+  // Optional list of extension pages that are sandboxed (served from a unique
+  // origin with a different Content Security Policy).
+  URLPatternSet pages;
+
+  // Content Security Policy that should be used to enforce the sandbox used
+  // by sandboxed pages (guaranteed to have the "sandbox" directive without the
+  // "allow-same-origin" token).
+  std::string content_security_policy;
+};
+
+class SandboxedHandler : public ManifestHandler {
+ public:
+  SandboxedHandler();
+  virtual ~SandboxedHandler();
+
+  virtual bool Parse(Extension* extension, string16* error) OVERRIDE;
+
+ private:
+  virtual const std::vector<std::string> Keys() const OVERRIDE;
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_COMMON_EXTENSIONS_SANDBOXED_HANDLER_H_