Bug 1244304 - Remove unused appId / inBrowser from permission manager. r=ehsan
--- a/extensions/cookie/nsPermissionManager.cpp
+++ b/extensions/cookie/nsPermissionManager.cpp
@@ -1768,24 +1768,16 @@ nsPermissionManager::AddInternal(nsIPrin
entry->GetPermissions()[index].mID = id;
entry->GetPermissions()[index].mPermission = aPermission;
entry->GetPermissions()[index].mExpireType = aExpireType;
entry->GetPermissions()[index].mExpireTime = aExpireTime;
entry->GetPermissions()[index].mModificationTime = aModificationTime;
// If requested, create the entry in the DB.
if (aDBOperation == eWriteToDB) {
- uint32_t appId;
- rv = aPrincipal->GetAppId(&appId);
- NS_ENSURE_SUCCESS(rv, rv);
-
- bool isInBrowserElement;
- rv = aPrincipal->GetIsInBrowserElement(&isInBrowserElement);
- NS_ENSURE_SUCCESS(rv, rv);
-
UpdateDB(eOperationAdding, mStmtInsert, id, origin, aType, aPermission,
aExpireType, aExpireTime, aModificationTime);
}
if (aNotifyOperation == eNotify) {
NotifyObserversWithPermission(aPrincipal,
mTypeArray[typeIndex],
aPermission,
@@ -2947,9 +2939,8 @@ nsPermissionManager::FetchPermissions()
// will end up as now()) is fine.
uint64_t modificationTime = 0;
AddInternal(principal, perm.type, perm.capability, 0, perm.expireType,
perm.expireTime, modificationTime, eNotify, eNoDBOperation,
true /* ignoreSessionPermissions */);
}
return NS_OK;
}
-