Closes mozilla-mobile#10231 update proton icons third batch

Amejia481 · May 12, 2021 · ff48f3e · ff48f3e
1 parent cc0a7f2
commit ff48f3e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/...in/java/mozilla/components/browser/engine/gecko/permission/GeckoSitePermissionsStorage.kt b/...in/java/mozilla/components/browser/engine/gecko/permission/GeckoSitePermissionsStorage.kt
@@ -46,6 +46,10 @@ class GeckoSitePermissionsStorage(
     private val geckoStorage: StorageController = runtime.storageController
     private val mainScope = CoroutineScope(Dispatchers.Main)
 
+    /*
+    * At the moment gecko doesn't handle temporary permission  see
+    * https://bugzilla.mozilla.org/show_bug.cgi?id=1710447
+    * */
     @VisibleForTesting
     internal val geckoTemporaryPermissions = mutableListOf<ContentPermission>()
 

diff --git a/...ssions/src/main/java/mozilla/components/feature/sitepermissions/SitePermissionsFeature.kt b/...ssions/src/main/java/mozilla/components/feature/sitepermissions/SitePermissionsFeature.kt
@@ -330,10 +330,10 @@ class SitePermissionsFeature(
                             status = status,
                             permissions = request.permissions
                         )
-                storage.save(sitePermissions)
+                storage.save(sitePermissions, request)
             } else {
                 sitePermissions = request.toSitePermissions(host, status, sitePermissions)
-                storage.update(sitePermissions)
+                storage.update(sitePermissions, request)
             }
         }
     }