select: fix maximum call stack size error and improve performance in multiple mode

[ogix]

These changes fix "Maximum call stack size exceeded" error when selecting lots of options by value (e.g. clicking on a button).
Also improve performance by checking which options have to be selected/deselected when a new value comes in. Currently SelectionModel is cleared and options are selected by one every time select value changes. This leads to poor performance when there are many options.

Fixes #12504

[crisbeto]

This is being handled by #12517 already.

[ogix]

@crisbeto I've seen your PR but I guess it only covers a case when you initialize with preselected options. When I was debugging select component code I noticed that 950 line calls this._selectionModel.select(option) again after it was called by _setSelectionByValue which causes Maximum call stack size exceeded error so I've added a check that checks if option selected value is different than in SelectionModel

[ogix]

After some testing I noticed that performance changes only improve performance when clicking on the "select all" button subsequently. It takes a bit longer on the first click. So I will close this PR and open another one only with recursive call fix.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}