refactor(drag-drop): remove deprecated APIs for 9.0.0

[crisbeto]

Cleans on the APIs that were marked as deprecated for 8.0 and 9.0. Also deprecates a few for 10.0 which were supposed to be removed in 9.0, but hadn't been marked correctly.

BREAKING CHANGES:

• DropListRef.id has been removed.
• CdkDragConfig has been replaced with DragRefConfig.
• DragDropRegistry.getDropContainer has been removed.
• CdkDropListContainer interface has been removed.
• CDK_DROP_LIST_CONTAINER has been replaced with CDK_DROP_LIST.
• CdkDrag.boundaryElementSelector has been replaced with CdkDrag.boundaryElement.
• _ngZone and _viewportRuler parameters in the DropListRef constructor are now required.
• distance parameter in DropListRef.drop is now required.

[jelbourn]

Should we have a migration for the renames?

[crisbeto]

I'm not sure how many people are actually going to hit them. The only documented public API that was renamed is boundaryElementSelector and even it's a regular property (not an Input).

[jelbourn]

Isn't it trivial, though, to just add it to our migration data? I would still do it if it's not any major effort

[crisbeto]

Looks like we don't have the schematics set up for v9. I've set them up in #17101.

[crisbeto]

Added schematics where possible.

[jelbourn]

LGTM

[devversion]

@crisbeto We should be able to add CDK_DROP_LIST_CONTAINER to the classNames data. I know it's technically not a class, but IIRC that is actually just doing migration for identifiers (can verify in a test case). Same for CdkDragConfig.

We can also handle distance parameter in DropListRef.drop is now required. in the method-calls-check data.

[crisbeto]

Nice, added 👍. The only part I'm confused by is what the invalidArgCounts.count means. Is it supposed to be the number of required parameters?

[devversion]

Yeah, I was confused about that rule at some point as well (i did not author it), but I guess it will print the given message if the arguments length matches the count length.

e.g. in that case you'd set it to the amount of params minus one (to match if the distance param is missing). In that case you need to set it to 4 I think.

[devversion]

LGTM schematic wise.

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