1849498 - Stop using InflatedChar16Sequence as parameter or temporary storage in ParserAtom

Status: ASSIGNED

(Core :: JavaScript Engine, task, P3)

Description

[Tooru Fujisawa [:arai]]

InflatedChar16Sequence is constructed with code units pointer and the length, and it stores the information as units pointer and the limit pointer.

https://searchfox.org/mozilla-central/rev/7d98e651953f3135d91e98fa6d33efa131aec7ea/js/src/util/Text.h#206-207

InflatedChar16Sequence(const CharT* units, size_t len)
    : units_(units), limit_(units_ + len) {}

Bug 1849490 is going to add optimized path which uses the length instead of limit pointer,
and storing limit pointer there results in unnecessary turnaround.

Also, sometimes the length is passed along with InflatedChar16Sequence, which is unnecessary.

It's better directly passing the code units pointer and length, for clarity, and simplicity.