Implement kiosk multiple apps feature.

extensions/common/manifest_handlers/kiosk_secondary_apps_info.cc

+// Copyright 2015 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 "extensions/common/manifest_handlers/kiosk_secondary_apps_info.h"
+
+#include "base/memory/scoped_ptr.h"
+#include "base/strings/string16.h"
+#include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
+#include "base/values.h"
+#include "extensions/common/manifest_constants.h"
+
+namespace extensions {
+
+namespace keys = manifest_keys;
+
+namespace {
+
+const char kKioskSecondaryAppInfo[] = "kiosk_secondary_app_info";
+const char kAppId[] = "id";
+}

[xiyuan, 2015/08/26 18:16:17]

nit: insert an empty line before. And append "// namespace".

[jennyz, 2015/08/28 18:24:08]

Done.

+
+KioskSecondaryAppsInfo::KioskSecondaryAppsInfo(
+    const std::vector<std::string>& app_ids)
+    : ids(app_ids) {}
+
+KioskSecondaryAppsInfo::~KioskSecondaryAppsInfo() {}
+
+// static
+bool KioskSecondaryAppsInfo::HaveSecondaryApps(const Extension* extension) {
+  return KioskSecondaryAppsInfo::Get(extension) != nullptr;
+}
+
+// static
+KioskSecondaryAppsInfo* KioskSecondaryAppsInfo::Get(
+    const Extension* extension) {
+  return static_cast<KioskSecondaryAppsInfo*>(
+      extension->GetManifestData(kKioskSecondaryAppInfo));
+}
+
+KioskSecondaryAppsHandler::KioskSecondaryAppsHandler() {}
+
+KioskSecondaryAppsHandler::~KioskSecondaryAppsHandler() {}
+
+bool KioskSecondaryAppsHandler::Parse(Extension* extension,
+                                      base::string16* error) {

[not at google - send to devlin, 2015/08/26 17:53:59]

It's easier to use a manifest type than hand-writing your own parsing:

https://code.google.com/p/chromium/codesearch#chromium/src/extensions/common/...

for example usage of one of these, see:

https://code.google.com/p/chromium/codesearch#chromium/src/extensions/common/...

It's less error prone, and gives you the opportunity to auto-generate
documentation.

[jennyz, 2015/08/28 18:24:08]

Yes, I applied the patch you suggested to
tools/json_schema_compiler/h_generator.py, it fixed the issue of header file of
the type missing FromValue function, but still it does not generate the parsing
code to wrap the top level vector type. So I called the
json_schema_compiler::util::PopulateArrayFromList to pass the list here. It
would be nice if this is supported by the utility script. 

+  // Kiosk secondary apps key is optional.
+  if (!extension->manifest()->HasKey(keys::kKioskSecondaryApps)) {
+    return true;
+  }
+
+  const base::Value* secondary_app_ids = nullptr;
+  if (!extension->manifest()->Get(keys::kKioskSecondaryApps,
+                                  &secondary_app_ids)) {
+    *error = base::ASCIIToUTF16(manifest_errors::kInvalidKioskSecondaryApps);
+    return false;
+  }
+
+  const base::ListValue* list = nullptr;
+  if (!secondary_app_ids->GetAsList(&list)) {
+    *error = base::ASCIIToUTF16(manifest_errors::kInvalidKioskSecondaryApps);
+    return false;
+  }
+
+  std::vector<std::string> app_ids;
+  for (size_t i = 0; i < list->GetSize(); ++i) {
+    const base::DictionaryValue* app_id_entry = nullptr;
+    if (!list->GetDictionary(i, &app_id_entry)) {
+      *error = base::ASCIIToUTF16(
+          manifest_errors::kInvalidKioskSecondaryAppsBadAppEntry);
+      return false;
+    }
+    std::string app_id;
+    if (!app_id_entry->GetStringWithoutPathExpansion(kAppId, &app_id)) {
+      *error = base::ASCIIToUTF16(
+          manifest_errors::kInvalidKioskSecondaryAppsBadAppId);
+      return false;
+    }
+    app_ids.push_back(app_id);
+  }
+  if (!app_ids.empty()) {
+    extension->SetManifestData(kKioskSecondaryAppInfo,
+                               new KioskSecondaryAppsInfo(app_ids));
+  }
+  return true;
+}
+
+const std::vector<std::string> KioskSecondaryAppsHandler::Keys() const {
+  return SingleKey(keys::kKioskSecondaryApps);
+}
+
+}  // namespace extensions