1768022 - Stop using `Cu.getGlobalForObject` to get the shared global

Status: RESOLVED FIXED

(Core :: XPConnect, task)

Description

[Tooru Fujisawa [:arai]]

Some code uses Cu.getGlobalForObject(this) to get the shared global of JSMs, but this can be achieved by globalThis

https://searchfox.org/mozilla-central/search?q=Cu.getGlobalForObject%28this%29%29&path=jsm&case=true&regexp=false

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Bug 1768022 - Use globalThis instead of Cu.getGlobalForObject(this). r?Standard8!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cddcdc46d348
Use globalThis instead of Cu.getGlobalForObject(this). r=Standard8,webdriver-reviewers,necko-reviewers,whimboo,kershaw

Comment 3

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/cddcdc46d348

Comment 4

[Alexandru Ionescu (needinfo me) [:alexandrui]]

== Change summary for alert #34170 (as of Thu, 19 May 2022 07:13:50 GMT) ==

Improvements:

Ratio	Test	Platform	Options	Absolute values (old vs new)
14%	tumblr FirstVisualChange	windows10-64-shippable-qr	fission warm webrender	192.93 -> 166.27
13%	tumblr FirstVisualChange	windows10-64-shippable-qr	fission warm webrender	190.25 -> 166.23
5%	tumblr ContentfulSpeedIndex	windows10-64-shippable-qr	fission warm webrender	471.04 -> 445.92
5%	tumblr SpeedIndex	windows10-64-shippable-qr	cold fission webrender	707.96 -> 672.08
5%	tumblr PerceptualSpeedIndex	windows10-64-shippable-qr	cold fission webrender	796.38 -> 757.92
...	...	...	...	...
3%	tumblr LastVisualChange	windows10-64-shippable-qr	fission warm webrender	856.52 -> 833.00

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=34170