1666983 - `SourceExtent.toStringEnd` isn't set to correct value when delazifying class constructor

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Tooru Fujisawa [:arai]]

function f() { class A { constructor() {} } } f()
function f() { class A { constructor() {} } new A() } f()

when delazifying f, the function contains class constructor,
when delazifying A,
class constructor's extent.toStringEnd is set to the end of constructor function itself, not the class.

we don't use the value, but it's better fixing it, or remove the filed in "update" struct.

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Bug 1666983 - Set ScriptStencil.extent.toStringEnd of class constructor to correct value when delazifying. r?tcampbell!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5a5bb14b79fe
Bug 1666983 - Set ScriptStencil.extent.toStringEnd of class constructor to correct value when delazifying. r=tcampbell

Comment 3

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b369792b033
Backed out changeset 5a5bb14b79fe for SM bustages on Stencil.cpp

Comment 4

[Tooru Fujisawa [:arai]]

Forgot to exclude mutable flags in FunctionFlags from the comparison.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/602f53c7db5f
Set ScriptStencil.extent.toStringEnd of class constructor to correct value when delazifying. r=tcampbell

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/602f53c7db5f