Port bug 1389529 |Remove deprecated nsIDOMStorageManager.getLocalStorageForPrincipal| to C-C

RESOLVED FIXED in Thunderbird 57.0

Status

RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jorgk, Assigned: frg)

Tracking

Trunk
Thunderbird 57.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

a year ago
mail/components/newmailaccount/content/accountProvisioner.js
return Services.domStorageManager.getLocalStorageForPrincipal(principal, url);

mail/test/mozmill/newmailaccount/test-newmailaccount.js
let storage = dsm.getLocalStorageForPrincipal(principal, url);

suite/common/dataman/dataman.js
lStorageForPrincipal(principal, "");
(Reporter)

Updated

a year ago
Summary: Port |Remove deprecated nsIDOMStorageManager.getLocalStorageForPrincipal| to C-C → Port bug 1389529 |Remove deprecated nsIDOMStorageManager.getLocalStorageForPrincipal| to C-C
(Assignee)

Comment 1

a year ago
nsIDOMStorageManager.getLocalStorageForPrincipal is only a thin wrapper for CreateStorage:

> comm-central/mozilla/dom/storage/LocalStorageManager.cpp
> NS_IMETHODIMP
> LocalStorageManager::GetLocalStorageForPrincipal(nsIPrincipal* aPrincipal,
>                                                  const nsAString& aDocumentURI,
>                                                  bool aPrivate,
>                                                  nsIDOMStorage** aRetval)
> {
>   return CreateStorage(nullptr, aPrincipal, aDocumentURI, aPrivate, aRetval);
> }
Assignee: nobody → frgrahl
Status: NEW → ASSIGNED
(Assignee)

Comment 2

a year ago
Created attachment 8896582 [details] [diff] [review]
1389673-createstorage-mail.patch

Patch for mail
Attachment #8896582 - Flags: review?(jorgk)
(Assignee)

Comment 3

a year ago
Created attachment 8896583 [details] [diff] [review]
1389673-createstorage.patch

Patch for suite. Could you review it too. Trivial patch, tested and if IanN minds he will tell me for sure later.
Attachment #8896583 - Flags: review?(jorgk)
(Reporter)

Comment 4

a year ago
Thanks, no rush here since bug 1389529 hasn't landed.
(Reporter)

Comment 5

a year ago
Comment on attachment 8896582 [details] [diff] [review]
1389673-createstorage-mail.patch

This must be right since the IDL says:
  /**
   * @deprecated
...
   * Currently just forwards to the createStorage method of this
   * interface.
...
  nsIDOMStorage getLocalStorageForPrincipal(in nsIPrincipal aPrincipal,
                                            in DOMString aDocumentURI,
                                            [optional] in bool aPrivate);
Attachment #8896582 - Flags: review?(jorgk) → review+
(Reporter)

Comment 6

a year ago
Comment on attachment 8896583 [details] [diff] [review]
1389673-createstorage.patch

Same comment as for the other patch.
Attachment #8896583 - Flags: review?(jorgk) → review+
(Reporter)

Updated

a year ago
Keywords: checkin-needed

Comment 7

a year ago
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/55f7f7d393e7
Port bug 1389529: Replace getLocalStorageForPrincipal with createStorage in mail. r=jorgk
https://hg.mozilla.org/comm-central/rev/412d334f32bc
Port bug 1389529: Replace getLocalStorageForPrincipal with createStorage in suite. r=jorgk
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Keywords: checkin-needed
Resolution: --- → FIXED
(Reporter)

Updated

a year ago
Target Milestone: --- → Thunderbird 57.0
You need to log in before you can comment on or make changes to this bug.