Switch platform apps from a declarative launch container to handling an onLaunched event.

chrome/common/extensions/extension_manifest_constants.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 "chrome/common/extensions/extension_manifest_constants.h"
 
 namespace extension_manifest_keys {
 
 const char kAllFrames[] = "all_frames";
 const char kAltKey[] = "altKey";
@@ skipping 123 matching lines @@
 const char kKeybindingPlatformLinux[] = "linux";
 const char kKeybindingPlatformMac[] = "mac";
 const char kKeybindingPlatformWin[] = "windows";
 const char kRunAtDocumentStart[] = "document_start";
 const char kRunAtDocumentEnd[] = "document_end";
 const char kRunAtDocumentIdle[] = "document_idle";
 const char kPageActionKeybindingEvent[] = "_execute_page_action";
 const char kPageActionTypeTab[] = "tab";
 const char kPageActionTypePermanent[] = "permanent";
 const char kLaunchContainerPanel[] = "panel";
-const char kLaunchContainerShell[] = "shell";
 const char kLaunchContainerTab[] = "tab";
 const char kLaunchContainerWindow[] = "window";
 }  // namespace extension_manifest_values
 
 // Extension-related error messages. Some of these are simple patterns, where a
 // '*' is replaced at runtime with a specific value. This is used instead of
 // printf because we want to unit test them and scanf is hard to make
 // cross-platform.
 namespace extension_manifest_errors {
 const char kAppsNotEnabled[] =
     "Apps are not enabled.";
 const char kBackgroundPermissionNeeded[] =
     "Hosted apps that use 'background_page' must have the 'background' "
     "permission.";
+const char kBackgroundRequiredForPlatformApps[] =
+    "Platform apps must have a background page or background scripts.";
 const char kCannotAccessPage[] =
     "Cannot access contents of url \"*\". "
     "Extension manifest must request permission to access this host.";
 const char kCannotChangeExtensionID[] =
     "Installed extensions cannot change their IDs.";
 const char kCannotClaimAllHostsInExtent[] =
     "Cannot claim all hosts ('*') in an extent.";
 const char kCannotClaimAllURLsInExtent[] =
     "Cannot claim all URLs in an extent.";
 const char kCannotScriptGallery[] =
@@ skipping 135 matching lines @@
 const char kInvalidKeyBindingDictionary[] =
     "Contents of 'commands[*]' invalid.";
 const char kInvalidKeyBindingMissingPlatform[] =
     "Could not find key specification for 'command[*].*': Either specify a key "
     "for '*', or specify a default key.";
 const char kInvalidKeyBindingUnknownPlatform[] =
     "Unknown platform for 'command[*]': *. Valid values are: 'windows', 'mac'"
     " 'chromeos', 'linux' and 'default'.";
 const char kInvalidLaunchContainer[] =
     "Invalid value for 'app.launch.container'.";
-const char kInvalidLaunchContainerForNonPlatform[] =
-    "'app.launch.container' = 'shell' can only be used with platform_app.";
-const char kInvalidLaunchContainerForPlatform[] =
-    "platform_app requires 'app.launch.container' == 'shell'.";
 const char kInvalidLaunchValue[] =
     "Invalid value for '*'.";
 const char kInvalidLaunchValueContainer[] =
     "Invalid container type for '*'.";
 const char kInvalidManifest[] =
     "Manifest file is invalid.";
 const char kInvalidManifestVersion[] =
     "The 'manifest_version' key must be present and set to 2 (without quotes).";
 const char kInvalidMatch[] =
     "Invalid value for 'content_scripts[*].matches[*]': *";
@@ skipping 107 matching lines @@
     "Invalid value for 'app.urls[*]': *";
 const char kInvalidWebURLs[] =
     "Invalid value for 'app.urls'.";
 const char kInvalidZipHash[] =
     "Required key 'zip_hash' is missing or invalid.";
 const char kLaunchPathAndExtentAreExclusive[] =
     "The 'app.launch.local_path' and 'app.urls' keys cannot both be set.";
 const char kLaunchPathAndURLAreExclusive[] =
     "The 'app.launch.local_path' and 'app.launch.web_url' keys cannot "
     "both be set.";
+const char kLaunchNotAllowedForPlatformApps[] =
+    "Platform apps may not have 'app.launch' keys.";
 const char kLaunchURLRequired[] =
     "Either 'app.launch.local_path' or 'app.launch.web_url' is required.";
 const char kLocalesMessagesFileMissing[] =
     "Messages file is missing for locale.";
 const char kLocalesNoDefaultLocaleSpecified[] =
     "Localization used, but default_locale wasn't specified in the manifest.";
 const char kLocalesNoDefaultMessages[] =
     "Default locale is defined but default data couldn't be loaded.";
 const char kLocalesNoValidLocaleNamesListed[] =
     "No valid locale name could be found in _locales directory.";
@@ skipping 17 matching lines @@
     "Loading platform_app extension type is turned off by default. "
     "You can enable this type with the --enable-platform-apps "
     "command-line flag.";
 const char kReservedMessageFound[] =
     "Reserved key * found in message catalog.";
 #if defined(OS_CHROMEOS)
 const char kIllegalPlugins[] =
     "Extensions cannot install plugins on Chrome OS";
 #endif
 }  // namespace extension_manifest_errors