Add the "options_ui" extension manifest field.

extensions/common/manifest_handlers/options_page_info.h

+// Copyright 2014 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 EXTENSIONS_COMMON_MANIFEST_HANDLERS_OPTIONS_UI_H_
+#define EXTENSIONS_COMMON_MANIFEST_HANDLERS_OPTIONS_UI_H_
+
+#include <string>
+#include <vector>
+
+#include "base/values.h"
+#include "extensions/common/extension.h"
+#include "extensions/common/manifest_handler.h"
+#include "url/gurl.h"
+
+namespace extensions {
+
+// A class to provide options page configuration settings from the manifest.
+class OptionsPageInfo : public Extension::ManifestData {
+ public:
+  OptionsPageInfo(const GURL& options_page,
+                  const GURL& options_ui_page,
+                  bool chrome_styles,
+                  bool open_in_tab);
+  virtual ~OptionsPageInfo();
+
+  // Returns the URL to the given extension's options page. This method supports
+  // both the "options_ui.page" field and the legacy "options_page" field. If
+  // both are present, it will return the value of "options_ui.page".
+  static const GURL& GetOptionsPage(const Extension* extension);
+
+  static bool HasOptionsPage(const Extension* extension);

[not at google - send to devlin, 2014/08/29 22:33:37]

Comment.

[ericzeng, 2014/08/29 23:54:04]

Done.

+
+  // Returns whether the Chrome user agent stylesheet should be applied to the
+  // given extension's options page.
+  static bool ShouldUseChromeStyle(const Extension* extension);
+
+  // Returns whether the given extension's options page should be opened in a
+  // new tab instead of an embedded popup.
+  static bool ShouldOpenInTab(const Extension* extension);
+
+  static scoped_ptr<OptionsPageInfo> Parse(

[not at google - send to devlin, 2014/08/29 22:33:37]

Oh hey. Now that I'm reading this in daylight, it should be called Create()
because it's creating a new OptionsPageInfo instance.

[ericzeng, 2014/08/29 23:54:04]

Done.

+      Extension* extension,
+      const base::Value* options_ui_value,
+      const std::string& options_page_string,
+      std::vector<InstallWarning>* install_warnings,
+      base::string16* error);
+
+ private:
+  bool LoadOptionsPage(const Extension* extension, base::string16* error);

[Yoyo Zhou, 2014/08/29 23:06:27]

This looks unused.

[ericzeng, 2014/09/02 18:23:37]

Done.

+
+  GURL options_page_;     // The legacy options_page entry
+  GURL options_ui_page_;  // The options_ui.page entry
+  bool chrome_styles_;
+  bool open_in_tab_;
+};
+
+// Parses the "options_ui" manifest key.
+class OptionsPageManifestHandler : public ManifestHandler {
+ public:
+  OptionsPageManifestHandler();
+  virtual ~OptionsPageManifestHandler();
+
+  virtual bool Parse(Extension* extension, base::string16* error) OVERRIDE;
+  virtual bool Validate(const Extension* extension,
+                        std::string* error,
+                        std::vector<InstallWarning>* warnings) const OVERRIDE;
+
+ private:
+  virtual const std::vector<std::string> Keys() const OVERRIDE;
+
+  DISALLOW_COPY_AND_ASSIGN(OptionsPageManifestHandler);
+};
+
+}  // namespace extensions
+
+#endif  // EXTENSIONS_COMMON_MANIFEST_HANDLERS_OPTIONS_UI_H_