Sorted kSafeManifestEntries array (cleanup only, no real changes)

chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc

 // Copyright 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.
 
 #include "chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.h"
 
 #include <stddef.h>
 
 #include <cstddef>
 #include <string>
@@ skipping 109 matching lines @@
     "oflckobdemeldmjddmlbaiaookhhcngo",  // Citrix Receiver QA version
     "ljacajndfccfgnfohlgkdphmbnpkjflk",  // Chrome Remote Desktop (Dev Build)
 };
 
 // List of manifest entries from https://developer.chrome.com/apps/manifest.
 // Unsafe entries are commented out and special cases too.
 const char* const kSafeManifestEntries[] = {
     // Special-cased in IsPlatformAppSafeForPublicSession().
     // emk::kApp,
 
-    // Special-cased in IsPlatformAppSafeForPublicSession().
-    // emk::kManifestVersion,
-
-    // Just a display string.
-    emk::kName,
-
-    // Just a display string.
-    emk::kShortName,
-
-    // Version string (for app updates).
-    emk::kVersion,
-
-    // Name of directory containg default strings.
-    emk::kDefaultLocale,
-
-    // An implementation detail (actually written by Chrome, not the app
-    // author).
-    emk::kCurrentLocale,
-
-    // Just a display string.
-    emk::kDescription,
-
-    // Just UX.
-    emk::kIcons,
-
     // Documented in https://developer.chrome.com/extensions/manifest but not
     // implemented anywhere.  Still, a lot of apps use it.
     "author",
 
     // TBD
     // emk::kBluetooth,
 
     // TBD
     // emk::kCommands,
 
+    // An implementation detail (actually written by Chrome, not the app
+    // author).
+    emk::kCurrentLocale,
+
+    // Name of directory containg default strings.
+    emk::kDefaultLocale,
+
+    // Just a display string.
+    emk::kDescription,
+
     // TBD, looks unsafe
     // emk::kEventRules,
 
+    // Shared Modules configuration: Allow other extensions to access resources.
+    emk::kExport,
+
     // TBD
     // emk::kExternallyConnectable,
 
     // TBD
     // emk::kFileHandlers,
 
     // TBD
     // emk::kFileSystemProviderCapabilities,
 
+    // Just UX.
+    emk::kIcons,
+
     // Shared Modules configuration: Import resources from another extension.
     emk::kImport,
 
-    // Shared Modules configuration: Allow other extensions to access resources.
-    emk::kExport,
-
     // Shared Modules configuration: Specify extension id for development.
     emk::kKey,
 
     // Descriptive statement about the app.
     emk::kKioskEnabled,
 
     // Contradicts the purpose of running inside a Public Session.
     // emk::kKioskOnly,
 
+    // Special-cased in IsPlatformAppSafeForPublicSession().
+    // emk::kManifestVersion,
+
     // Descriptive statement about the app.
     emk::kMinimumChromeVersion,
 
     // NaCl modules are bound to app permissions just like the rest of the app
     // and thus should not pose a risk.
     emk::kNaClModules,
 
+    // Just a display string.
+    emk::kName,
+
     // TBD, doc missing
     // emk::kOAuth2,
 
     // Descriptive statement about the app.
     emk::kOfflineEnabled,
 
     // Special-cased in IsPlatformAppSafeForPublicSession().
     // emk::kOptionalPermissions,
 
     // Special-cased in IsPlatformAppSafeForPublicSession().
     // emk::kPermissions,
 
     // No constant in manifest_constants.cc.
     // "platforms",
 
     // Descriptive statement about the app.
     emk::kRequirements,
 
     // Execute some pages in a separate sandbox.  (Note: Using string literal
     // since extensions::manifest_keys only has constants for sub-keys.)
     "sandbox",
 
+    // Just a display string.
+    emk::kShortName,
+
     // TBD, doc missing
     // emk::kSignature,
 
     // Network access.
     emk::kSockets,
 
     // TBD.  (Note: Using string literal since extensions::manifest_keys only
     // has constants for sub-keys.)
     // "storage",
 
     // TBD, doc missing
     // emk::kSystemIndicator,
 
     // TODO(tnagel): Ensure that extension updates query UserMayLoad().
     // https://crbug.com/549720
     emk::kUpdateURL,
 
     // Apps may intercept navigations to URL patterns for domains for which the
     // app author has proven ownership of to the Web Store.  (Chrome starts the
     // app instead of fulfilling the navigation.)  This is only safe for apps
     // that have been loaded from the Web Store and thus is special-cased in
     // IsPlatformAppSafeForPublicSession().
     // emk::kUrlHandlers,
 
     // TBD
     // emk::kUsbPrinters,
 
+    // Version string (for app updates).
+    emk::kVersion,
+
     // Just a display string.
     emk::kVersionName,
 
     // Webview has no special privileges or capabilities.
     emk::kWebview,
 };
 
 // List of permission strings based on [1] and [2].  See |kSafePermissionDicts|
 // for permission dicts.  Since Public Session users may be fully unaware of any
 // apps being installed, their consent to access any kind of sensitive
@@ skipping 360 matching lines @@
   if (error) {
     *error = l10n_util::GetStringFUTF16(
           IDS_EXTENSION_CANT_INSTALL_IN_DEVICE_LOCAL_ACCOUNT,
           base::UTF8ToUTF16(extension->name()),
           base::UTF8ToUTF16(extension->id()));
   }
   return false;
 }
 
 }  // namespace chromeos