fix(material-experimental/chips): remove icon responds to invalid keys

[vanessanschmitt]

Fixes an issue where pressing any non-arrow key while the chip
remove icon is focused removes the chip. This should only happen for
ENTER or SPACE. This issue was introduced when MDC refactored the
chip foundation. The handleTrailingIconInteraction method, which
used to call shouldHandleInteraction to detect if the keydown event
was ENTER or SPACE, was replaced with handleTrailingActionInteraction,
which handles all keydown events. This change was made in
github.com/material-components/material-components-web/pull/5890, and
MDC chips were migrated in github.com//pull/19318.

[mmalerba]

LGTM

[crisbeto]

There's a pull request for this already #19425.

[vanessanschmitt]

There's a pull request for this already #19425.

Oh I didn't see this, thank you for pointing it out! @mmalerba Should I abandon this PR so you can approve that one instead? We need this fix for Pantheon but both PRs do the same thing and both add tests, so I'm not tied to this one.

[mmalerba]

Ok, I've bumped the priority on that one to match this one and approved it. Closing this one

[vanessanschmitt]

Thank you!

[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.}