1688202 - Use TypeDescr instead JSObject in ObjectGroupRealm and remove more unused functions

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task)

Description

[André Bargull [:anba]]

• Use TypeDescr instead JSObject in ObjectGroupRealm.
• Update some ObjectGroup comments.
• And remove more unused functions.

Comment 1

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 1: Use TypeDescr for ObjectGroupRealm. r=jandem!

The associated object is only used for typed objects, so we can change it to
TypeDescr.

Comment 2

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 2: Remove no longer needed assertions in ObjectGroupRealm::NewEntry::Lookup. r=jandem!

After part 1 it's easy to see why this assertion is no longer needed.

Depends on D102740

Comment 3

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 3: Remove unused ObjectGroup functions. r=jandem!

Remove unused ObjectGroup functions and also remove a no longer forward
declaration of gc::MergeRealms.

Drive-by change:

• Add missing include for gc::MergeRealms to "HelperThreads.cpp".

Depends on D102741

Comment 4

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 4: Update some ObjectGroup comments. r=jandem!

Remove comments about "type properties" etc. which were refering to TI features.

Depends on D102742

Comment 5

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 5: Remove unused NativeObject methods. r=jandem!

Depends on D102743

Comment 6

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 6: Remove ReshapeForAllocKind. r=jandem!

Depends on D102745

Comment 7

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 7: Remove RegExpStatics::updateLazily. r=jandem!

Depends on D102746

Comment 8

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 8: Remove RegExpObject::create() variants accepting TokenStreamAnyChars. r=jandem!

Depends on D102747

Comment 9

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 9: Remove unused ReceiverGuard methods. r=jandem!

Depends on D102748

Comment 10

[André Bargull [:anba]]

Attachment: Bug 1688202 - Part 10: Remove unused JSFunction functions. r=jandem!

Depends on D102749

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7859910469d5
Part 1: Use TypeDescr for ObjectGroupRealm. r=jandem
https://hg.mozilla.org/integration/autoland/rev/60c2d0fe83ae
Part 2: Remove no longer needed assertions in ObjectGroupRealm::NewEntry::Lookup. r=jandem
https://hg.mozilla.org/integration/autoland/rev/d6fd42dc5f0f
Part 3: Remove unused ObjectGroup functions. r=jandem
https://hg.mozilla.org/integration/autoland/rev/84b74cd250f9
Part 4: Update some ObjectGroup comments. r=jandem
https://hg.mozilla.org/integration/autoland/rev/44b5e468df6a
Part 5: Remove unused NativeObject methods. r=jandem
https://hg.mozilla.org/integration/autoland/rev/75e1e508676f
Part 6: Remove ReshapeForAllocKind. r=jandem
https://hg.mozilla.org/integration/autoland/rev/84fe88c4d7cf
Part 7: Remove RegExpStatics::updateLazily. r=jandem
https://hg.mozilla.org/integration/autoland/rev/94e14cf6985e
Part 8: Remove RegExpObject::create() variants accepting TokenStreamAnyChars. r=jandem
https://hg.mozilla.org/integration/autoland/rev/1ebb9974ea0b
Part 9: Remove unused ReceiverGuard methods. r=jandem
https://hg.mozilla.org/integration/autoland/rev/cf536060cd46
Part 10: Remove unused JSFunction functions. r=jandem

Comment 12

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7859910469d5
https://hg.mozilla.org/mozilla-central/rev/60c2d0fe83ae
https://hg.mozilla.org/mozilla-central/rev/d6fd42dc5f0f
https://hg.mozilla.org/mozilla-central/rev/84b74cd250f9
https://hg.mozilla.org/mozilla-central/rev/44b5e468df6a
https://hg.mozilla.org/mozilla-central/rev/75e1e508676f
https://hg.mozilla.org/mozilla-central/rev/84fe88c4d7cf
https://hg.mozilla.org/mozilla-central/rev/94e14cf6985e
https://hg.mozilla.org/mozilla-central/rev/1ebb9974ea0b
https://hg.mozilla.org/mozilla-central/rev/cf536060cd46