Fix for broken "Load unpacked extension" popup

chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc

Index: chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
diff --git a/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc b/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
index 28af4c8469ba2a3acf05fc87be106e56fc9c6ea0..341867fcb3f566032942b609a19f0408716780a2 100644
--- a/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
+++ b/chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.cc
@@ -34,9 +34,10 @@ void PermissionsUpdaterDelegateChromeOS::InitializePermissions(
   // the paste (the end result is just an empty paste).
   APIPermissionSet api_permission_set((*granted_permissions)->apis());
   api_permission_set.erase(APIPermission::kClipboardRead);
-  granted_permissions->reset(
-      new PermissionSet(api_permission_set, ManifestPermissionSet(),
-                        URLPatternSet(), URLPatternSet()));
+  granted_permissions->reset(new PermissionSet(
+      api_permission_set, (*granted_permissions)->manifest_permissions(),

[Ivan Šandrk, 2017/04/25 18:51:59]

Should I maybe have a temporary helper variable to shorten the following?:

(*granted_permissions)->...

[Devlin, 2017/04/25 18:57:48]

I'd say it's probably not worth it, since this is resetting the granted
permissions, so a variable set to the old value would become unsafe very soon. 
But, if you wanted to, you could scope it, e.g.
api_permission_set.erase(APIPermission::kClipboardRead);
{
  const PermissionSet& granted = *granted_permissions;
  granted_permissions = base::MakeUnique<PermissionSet>(
      api_permission_set, granted.manifest_permissions(),
      granted.explicit_hosts(), granted.scriptable_hosts()));
}

which would discourage its unsafe use and would be okay.

[Ivan Šandrk, 2017/04/25 19:42:11]

I guess it's not worth the bother.

+      (*granted_permissions)->explicit_hosts(),
+      (*granted_permissions)->scriptable_hosts()));
 }
 
 }  // namespace extensions