Fix ExtensionSpecialStoragePolicy.IsInstalledApp

chrome/browser/extensions/extension_special_storage_policy.cc

Index: chrome/browser/extensions/extension_special_storage_policy.cc
diff --git a/chrome/browser/extensions/extension_special_storage_policy.cc b/chrome/browser/extensions/extension_special_storage_policy.cc
index 4017d034bbb258ade245a6c40da6b39839deebca..6afd242a921adee43565f099299c56a28716166b 100644
--- a/chrome/browser/extensions/extension_special_storage_policy.cc
+++ b/chrome/browser/extensions/extension_special_storage_policy.cc
@@ -91,7 +91,8 @@ void ExtensionSpecialStoragePolicy::GrantRightsForExtension(
       !extension->HasAPIPermission(
           APIPermission::kUnlimitedStorage) &&
       !extension->HasAPIPermission(
-          APIPermission::kFileBrowserHandler)) {
+          APIPermission::kFileBrowserHandler) &&
+      !extension->is_app()) {

[asargent_no_longer_on_chrome, 2013/04/15 18:43:11]

nit: it would be good to keep the order of tests above here the same as the
order of tests inside the autolock'ed code below. E.g. protection, app,
unlimited permission, file browser handler. 

also optional suggestion: would this block be easier to read if it  was written
like this:

  if (!(A || B || C))

instead of like this:

  if (!A && !B && !C)

?

[kinuko, 2013/04/16 07:20:13]

Done.

     return;
   }
   {