Update extension install prompt to reflect withheld permissions

chrome/browser/extensions/permissions_updater.cc

 // Copyright (c) 2012 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/extensions/permissions_updater.h"
 
 #include "base/json/json_writer.h"
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 146 matching lines @@
   if (!Manifest::IsUnpackedLocation(extension->location()) &&
       extension->location() != Manifest::INTERNAL)
     return;
 
   ExtensionPrefs::Get(browser_context_)->AddGrantedPermissions(
       extension->id(),
       extension->permissions_data()->active_permissions().get());
 }
 
 void PermissionsUpdater::InitializePermissions(const Extension* extension) {
-  scoped_refptr<const PermissionSet> active_permissions =
-      ExtensionPrefs::Get(browser_context_)
-          ->GetActivePermissions(extension->id());
-  scoped_refptr<const PermissionSet> bounded_active =
-      GetBoundedActivePermissions(extension, active_permissions);
+  InitializePermissions(extension, INIT_FLAG_NONE);
+}
+
+void PermissionsUpdater::InitializePermissions(const Extension* extension,
+                                               InitFlag init_flag) {
+  scoped_refptr<const PermissionSet> active_permissions(NULL);
+  scoped_refptr<const PermissionSet> bounded_active(NULL);
+  // If |extension| is a transient dummy extension, we do not want to look for
+  // it in preferences.
+  if (init_flag & INIT_FLAG_TRANSIENT) {
+    bounded_active = active_permissions =
+        extension->permissions_data()->active_permissions();
+  } else {
+    active_permissions = ExtensionPrefs::Get(browser_context_)
+                             ->GetActivePermissions(extension->id());
+    bounded_active = GetBoundedActivePermissions(extension, active_permissions);
+  }
 
   // Withhold permissions only if the switch applies to this extension and the
   // extension doesn't have the preference to allow scripting on all urls.
   bool should_withhold_permissions =
       util::ScriptsMayRequireActionForExtension(extension) &&
       !util::AllowedScriptingOnAllUrls(extension->id(), browser_context_);
 
   URLPatternSet granted_explicit_hosts;
   URLPatternSet withheld_explicit_hosts;
   SegregateUrlPermissions(bounded_active->explicit_hosts(),
@@ skipping 25 matching lines @@
   bounded_active = new PermissionSet(bounded_active->apis(),
                                      bounded_active->manifest_permissions(),
                                      granted_explicit_hosts,
                                      granted_scriptable_hosts);
 
   scoped_refptr<const PermissionSet> withheld =
       new PermissionSet(APIPermissionSet(),
                         ManifestPermissionSet(),
                         withheld_explicit_hosts,
                         withheld_scriptable_hosts);
-  SetPermissions(extension, bounded_active, withheld);
+
+  if (init_flag & INIT_FLAG_TRANSIENT)
+    SetPermissionsWithoutPrefs(extension, bounded_active, withheld);
+  else
+    SetPermissions(extension, bounded_active, withheld);
 }
 
 void PermissionsUpdater::WithholdImpliedAllHosts(const Extension* extension) {
   scoped_refptr<const PermissionSet> active =
       extension->permissions_data()->active_permissions();
   scoped_refptr<const PermissionSet> withheld =
       extension->permissions_data()->withheld_permissions();
 
   URLPatternSet withheld_scriptable = withheld->scriptable_hosts();
   URLPatternSet active_scriptable;
@@ skipping 55 matching lines @@
     const Extension* extension,
     const scoped_refptr<const PermissionSet>& active,
     scoped_refptr<const PermissionSet> withheld) {
   withheld = withheld.get() ? withheld
                  : extension->permissions_data()->withheld_permissions();
   extension->permissions_data()->SetPermissions(active, withheld);
   ExtensionPrefs::Get(browser_context_)->SetActivePermissions(
       extension->id(), active.get());
 }
 
+void PermissionsUpdater::SetPermissionsWithoutPrefs(
+    const Extension* extension,
+    const scoped_refptr<const PermissionSet>& active,
+    scoped_refptr<const PermissionSet> withheld) {
+  withheld = withheld.get()
+                 ? withheld
+                 : extension->permissions_data()->withheld_permissions();
+  extension->permissions_data()->SetPermissions(active, withheld);
+}
+
 void PermissionsUpdater::DispatchEvent(
     const std::string& extension_id,
     const char* event_name,
     const PermissionSet* changed_permissions) {
   EventRouter* event_router = EventRouter::Get(browser_context_);
   if (!event_router)
     return;
 
   scoped_ptr<base::ListValue> value(new base::ListValue());
   scoped_ptr<api::permissions::Permissions> permissions =
@@ skipping 47 matching lines @@
             Profile::FromBrowserContext(host->GetBrowserContext()))) {
       host->Send(new ExtensionMsg_UpdatePermissions(params));
     }
   }
 
   // Trigger the onAdded and onRemoved events in the extension.
   DispatchEvent(extension->id(), event_name, changed);
 }
 
 }  // namespace extensions