Closed Bug 1691008 Opened 3 months ago Closed 3 months ago

Remove left-over references to singleton objects

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
87 Branch
Tracking Status
firefox87 --- fixed

People

(Reporter: anba, Assigned: anba)

Details

Attachments

(5 files)

Remove left-over comments for singleton objects and replace callers to Wrapper::NewSingleton with Wrapper::New

Drive-by change:

  • Remove unnecessary using statements.

Depends on D104181

Drive-by change:

  • Fix some typos in ReshapeForProtoMutation().

Depends on D104183

Wrapper::NewSingleton() performs the same operations as Wrapper::New().

Depends on D104184

Severity: -- → N/A
Priority: -- → P2
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/9c067e33e062
Part 1: Remove CreateSingleton enum. r=jandem
https://hg.mozilla.org/integration/autoland/rev/2567f9e7a5a3
Part 2: Remove reference to "Singleton" in PlainObject.cpp. r=jandem
https://hg.mozilla.org/integration/autoland/rev/104c7648f1b7
Part 3: Remove SingletonObjectLiteralOperation declaration. r=jandem
https://hg.mozilla.org/integration/autoland/rev/cec65ae7ba08
Part 4: Remove reference to "Singleton" in setUncacheableProto(). r=jandem
https://hg.mozilla.org/integration/autoland/rev/b3580d2200b2
Part 5: Remove Wrapper::NewSingleton. r=jandem
You need to log in before you can comment on or make changes to this bug.