1992437 - Move XPCOMUtils.defineLazy/declareLazy to ChromeUtils

Status: ASSIGNED

(Core :: XPConnect, task)

Description

[Mark Banner (:standard8)]

We originally added XPCOMUtils.defineLazy/declareLazy in bug 1945464 as a way of prototyping them and testing out their use.

We are starting to use them in various places to aid with TypeScript integration, and hence we are having to re-include XPCOMUtils in various places.

In the review of the patches, it was suggested that we should move these implementations to C++, i.e. via ChromeUtils, to be more efficient.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1992437 - Port XPCOMUtils.defineLazy to ChromeUtils. r?arai!,#dom-core!

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1992437 - Port XPCOMUtils.declareLazy to ChromeUtils. r?arai!,#dom-core!

Comment 3

[Mark Banner (:standard8)]

Attachment: WIP: - Bug 1992437 - Migrate uses of XPCOMUtils.declareLazy to ChromeUtils.declareLazy.

Comment 4

[Mark Banner (:standard8)]

Attachment: WIP: - Bug 1992437 - Remove XPCOMUtils.declareLazy and XPCOMUtils.defineLazy.

Comment 5

[Mark Banner (:standard8)]

Attachment: Bug 1992437 - Explore using more webidl types in lazy preference getters. r?arai!,smaug!

Exploration of using webidl types as much as possible in the lazy preference getters. We still can't transition more to the webidl, as the lazy getters are functions that are injected onto JavaScript objects.

If we want to go ahead with this, then I'll fold it into D300779 before landing. I've kept it separate for now in case we don't want to go this far.