Add "kiosk_only" manifest attribute for platform apps.

chrome/common/extensions/manifest_handlers/kiosk_mode_info.h

 // 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_MANIFEST_HANDLERS_KIOSK_ENABLED_INFO_H_
-#define CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_KIOSK_ENABLED_INFO_H_
+#ifndef CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_KIOSK_MODE_INFO_H_
+#define CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_KIOSK_MODE_INFO_H_
 
 #include <string>
 #include <vector>
 
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/manifest_handler.h"
 #include "extensions/common/manifest.h"
 
 namespace extensions {
 
 struct KioskEnabledInfo : public Extension::ManifestData {
   explicit KioskEnabledInfo(bool is_kiosk_enabled);
   virtual ~KioskEnabledInfo();
 
   bool kiosk_enabled;
 
   // Whether the extension or app should be enabled in app kiosk mode.
   static bool IsKioskEnabled(const Extension* extension);
 };
 

[not at google - send to devlin, 2013/09/24 18:35:17]

consider replacing these with an enum:

// something like this:
struct KioskModeInfo {
  enum Value {
    NONE,
    ENABLED,
    ONLY,
  }

  ...
};

and enforcing that safety at parse time.

[Tim Song, 2013/09/24 18:50:01]

I considered this, but an issue arises when calling Extension::SetManifestData.
If we replace both of these fields with one struct, then we lose the symmetry of
fields between the manifest json and the internal data representation.

[not at google - send to devlin, 2013/09/24 18:55:17]

I think this is fine; I CC'ed Yoyo who is the manifest handler person. Safe
internal model is better than making call sites figure it out.

[Tim Song, 2013/09/24 19:09:06]

Alright, then we'll just have one ManifestData for the two fields keyed by
keys::kKioskMode.

-// Parses the "kiosk_enabled" manifest key.
-class KioskEnabledHandler : public ManifestHandler {
+struct KioskOnlyInfo : public Extension::ManifestData {
+  explicit KioskOnlyInfo(bool is_kiosk_only);
+  virtual ~KioskOnlyInfo();
+
+  bool kiosk_only;
+
+  // Whether the extension or app should only be available in kiosk mode.
+  static bool IsKioskOnly(const Extension* extension);
+};
+
+// Parses the "kiosk_enabled" and "kiosk_only" manifest keys.
+class KioskModeHandler : public ManifestHandler {
  public:
-  KioskEnabledHandler();
-  virtual ~KioskEnabledHandler();
+  KioskModeHandler();
+  virtual ~KioskModeHandler();
 
   virtual bool Parse(Extension* extension, string16* error) OVERRIDE;
+
  private:
   virtual const std::vector<std::string> Keys() const OVERRIDE;
 
-  DISALLOW_COPY_AND_ASSIGN(KioskEnabledHandler);
+  std::vector<std::string> supported_keys_;
+
+  DISALLOW_COPY_AND_ASSIGN(KioskModeHandler);
 };
 
 }  // namespace extensions
 
-#endif  // CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_KIOSK_ENABLED_INFO_H_
+#endif  // CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_KIOSK_MODE_INFO_H_