Restrict use of two app-launching command line flags

apps/app_load_service.cc

Index: apps/app_load_service.cc
diff --git a/apps/app_load_service.cc b/apps/app_load_service.cc
index eb4b5c6b1c5e7231fe7bd59986da09a0b808b3c0..bf04d2d73649d7b4a87a95de22692bcc72a1d7b3 100644
--- a/apps/app_load_service.cc
+++ b/apps/app_load_service.cc
@@ -64,8 +64,9 @@ bool AppLoadService::LoadAndLaunch(const base::FilePath& extension_path,
   ExtensionService* extension_service =
       ExtensionSystem::Get(profile_)->extension_service();
   std::string extension_id;
-  if (!extensions::UnpackedInstaller::Create(extension_service)->
-          LoadFromCommandLine(base::FilePath(extension_path), &extension_id)) {
+  if (!extensions::UnpackedInstaller::Create(extension_service)
+           ->LoadFromCommandLine(base::FilePath(extension_path), &extension_id,
+                                 true)) {
     return false;
   }
 
@@ -81,8 +82,9 @@ bool AppLoadService::Load(const base::FilePath& extension_path) {
   ExtensionService* extension_service =
       ExtensionSystem::Get(profile_)->extension_service();
   std::string extension_id;
-  return extensions::UnpackedInstaller::Create(extension_service)->
-      LoadFromCommandLine(base::FilePath(extension_path), &extension_id);
+  return extensions::UnpackedInstaller::Create(extension_service)
+      ->LoadFromCommandLine(base::FilePath(extension_path), &extension_id,
+                            true);

[asargent_no_longer_on_chrome, 2016/07/20 21:33:32]

optional: sometimes it can aid readability to either use an enum for paramters
like this (eg ONLY_ALLOW_APPS, ALLOW_ALL_TYPES), or at least have an inline
comment about the meaning of the boolean, so that the reader doesn't have to go
look at the declaration of the method in some other file to understand what the
meaning is - e.g.

LoadFromCommandLine(base::FilePath(extension_path), &extension_id,
                    true /* only_allow_apps */);

[proberge, 2016/07/21 14:59:43]

Done.

 }
 
 // static