Fix memory leak in ActiveTabPermissionManager.

chrome/common/extensions/extension.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/common/extensions/extension.h"
 
 #include <ostream>
 
 #include "base/base64.h"
 #include "base/basictypes.h"
@@ skipping 3811 matching lines @@
 }
 
 scoped_refptr<const PermissionSet> Extension::GetTabSpecificPermissions(
     int tab_id) const {
   base::AutoLock auto_lock(runtime_data_lock_);
   return runtime_data_.GetTabSpecificPermissions(tab_id);
 }
 
 void Extension::UpdateTabSpecificPermissions(
     int tab_id,
-    const PermissionSet* permissions) const {
+    scoped_refptr<const PermissionSet> permissions) const {
   base::AutoLock auto_lock(runtime_data_lock_);
   runtime_data_.UpdateTabSpecificPermissions(tab_id, permissions);
 }
 
 void Extension::ClearTabSpecificPermissions(int tab_id) const {
   base::AutoLock auto_lock(runtime_data_lock_);
   runtime_data_.ClearTabSpecificPermissions(tab_id);
 }
 
 bool Extension::CheckPlatformAppFeatures(std::string* utf8_error) {
@@ skipping 37 matching lines @@
 
 scoped_refptr<const PermissionSet>
     Extension::RuntimeData::GetTabSpecificPermissions(int tab_id) const {
   CHECK_GE(tab_id, 0);
   TabPermissionsMap::const_iterator it = tab_specific_permissions_.find(tab_id);
   return (it != tab_specific_permissions_.end()) ? it->second : NULL;
 }
 
 void Extension::RuntimeData::UpdateTabSpecificPermissions(
     int tab_id,
-    const PermissionSet* permissions) {
+    scoped_refptr<const PermissionSet> permissions) {
   CHECK_GE(tab_id, 0);
   if (tab_specific_permissions_.count(tab_id)) {
     tab_specific_permissions_[tab_id] = PermissionSet::CreateUnion(
         tab_specific_permissions_[tab_id],
-        permissions);
+        permissions.get());
   } else {
     tab_specific_permissions_[tab_id] = permissions;
   }
 }
 
 void Extension::RuntimeData::ClearTabSpecificPermissions(int tab_id) {
   CHECK_GE(tab_id, 0);
   tab_specific_permissions_.erase(tab_id);
 }
 
 UnloadedExtensionInfo::UnloadedExtensionInfo(
     const Extension* extension,
     extension_misc::UnloadedExtensionReason reason)
   : reason(reason),
     already_disabled(false),
     extension(extension) {}
 
 UpdatedExtensionPermissionsInfo::UpdatedExtensionPermissionsInfo(
     const Extension* extension,
     const PermissionSet* permissions,
     Reason reason)
     : reason(reason),
       extension(extension),
       permissions(permissions) {}
 
 }   // namespace extensions