Closed Bug 1773797 Opened 2 years ago Closed 2 years ago

Convert all instance of do_CreateInstance for nsICryptoHash with NS_NewCryptoHash

Categories

(Core :: XPCOM, task)

task

Tracking

()

RESOLVED FIXED
103 Branch
Tracking Status
firefox103 --- fixed

People

(Reporter: barret, Assigned: barret)

References

Details

Attachments

(1 file)

Now that we have a C++ function to construct nsICryptoHash instances directly, we don't need to incur the run-time cost of calling do_CreateInstance("@mozilla.org/security/hash;1").

There are only a few usages:

	dom/media/gmp/CDMStorageIdProvider.cpp
37	do_CreateInstance("@mozilla.org/security/hash;1", &rv);
	dom/security/SRICheck.cpp
205	do_CreateInstance("@mozilla.org/security/hash;1", &rv);
	dom/security/nsCSPUtils.cpp
990	hasher = do_CreateInstance("@mozilla.org/security/hash;1", &rv);
	toolkit/components/reputationservice/ApplicationReputation.cpp
1297	do_CreateInstance("@mozilla.org/security/hash;1", &rv);
Pushed by brennie@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e6471acf5208
Replace do_CreateInstance for nsICryptoHash with NS_NewCryptoHash r=Gijs,media-playback-reviewers,padenot,nika,ckerschb
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Regressions: 1775119
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: