Bug: sorting in ngoninit causes crash

[swftvsn]

sort.js:538 Uncaught TypeError: Cannot read property 'toState' of undefined
at SafeSubscriber._next (sort.js:538)
at SafeSubscriber.__tryOrUnsub (Subscriber.js:183)
at SafeSubscriber.next (Subscriber.js:122)
at Subscriber._next (Subscriber.js:72)
at Subscriber.next (Subscriber.js:49)
at MergeMapSubscriber.notifyNext (mergeMap.js:70)
at SimpleInnerSubscriber._next (innerSubscribe.js:10)
at SimpleInnerSubscriber.next (Subscriber.js:49)
at SafeSubscriber.schedulerFn [as _next] (core.js:25851)
at SafeSubscriber._tryOrUnsub (Subscriber.js:183)
at SafeSubscriber.next (Subscriber.js:122)
at Subscriber.next (Subscriber.js:72)
at Subscriber.next (Subscriber.js:49)
at EventEmitter.next (Subject.js:39)
at EventEmitter.emit (core.js:25841)
at MatSort.sort (sort.js:139)

[swftvsn]

This also makes it happen: matSort matSortActive="name" matSortDirection="asc" (in html template)

[swftvsn]

Here's a screenshot of the offending line:

[swftvsn]

The commit that broke this was:
ca7d379

The new code fails to check if viewState exists. This is explained in this comment: ca7d379#r48262154

[swftvsn]

There's a PR for this here: #22228

[swftvsn]

Aaand second PR in #22229 Still failed to deliver, it seems that my code went over the 100 char limit.

[swftvsn]

Third time, let's see how it goes. Please take a look here: #22355

[aailyin]

@crisbeto Could you prioritize this bug, please? I see that PR with the fix is already opened.

[crisbeto]

We can't merge the open PR, because the CI checks are failing.

[chunghha]

@crisbeto, I just opened #22634 let's see if it passes.