Restrict use of two app-launching command line flags

chrome/browser/extensions/unpacked_installer.h

Index: chrome/browser/extensions/unpacked_installer.h
diff --git a/chrome/browser/extensions/unpacked_installer.h b/chrome/browser/extensions/unpacked_installer.h
index 2bdc85eb931dbfbad8b2bc745a0b49060354461d..602ec6eb354b5d4c1bbcaa640dc830197439fa31 100644
--- a/chrome/browser/extensions/unpacked_installer.h
+++ b/chrome/browser/extensions/unpacked_installer.h
@@ -51,7 +51,8 @@ class UnpackedInstaller
   // The return value indicates whether the installation has begun successfully.
   // The id of the extension being loaded is returned in |extension_id|.
   bool LoadFromCommandLine(const base::FilePath& extension_path,
-                           std::string* extension_id);
+                           std::string* extension_id,
+                           bool only_allow_apps);

[Devlin, 2016/06/29 14:59:54]

nit: I think this would be more flexible if we passed in a
std::set<Manifest::Type> allowed_types (perhaps making it a ptr so that null
implied all types).

[proberge, 2016/06/29 20:56:49]

With only three call sites, this seems a bit overkill. I'll keep the bool unless
the other reviewers insist.

[Devlin, 2016/06/29 21:04:15]

I imagine we'll want to make a similar change to the load-extension switch. We
can wait until that comes about, but it does seem like now would be a better
time.

 
   // Allows prompting for plugins to be disabled; intended for testing only.
   bool prompt_for_plugins() { return prompt_for_plugins_; }