fix currentCookieStoreId left undefined #1312

mozilla · Dec 9, 2019 · 347a057 · 347a057
1 parent e44fcdf
commit 347a057
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/js/background/assignManager.js b/src/js/background/assignManager.js
@@ -490,6 +490,9 @@ const assignManager = {
         currentCookieStoreId = backgroundLogic.cookieStoreId(currentUserContextId);
         confirmUrl += `&currentCookieStoreId=${currentCookieStoreId}`;
       }
+      else {
+        currentCookieStoreId = "firefox-default";
+      }
       browser.tabs.create({
         url: confirmUrl,
         cookieStoreId: currentCookieStoreId,

diff --git a/test/features/assignment.test.js b/test/features/assignment.test.js
@@ -34,7 +34,7 @@ describe("Assignment Feature", () => {
           url: "moz-extension://multi-account-containers/confirm-page.html?" +
                `url=${encodeURIComponent(activeTab.url)}` +
                `&cookieStoreId=${activeTab.cookieStoreId}`,
-          cookieStoreId: undefined,
+          cookieStoreId: "firefox-default",
           openerTabId: null,
           index: 2,
           active: true