fix(tabs): no longer use OnPush

[jelbourn]

This resolves an issue with ivy where the template content would be
checked out of sync with the tab view. See #15440 for additional
context.

[kara]

LGTM

[ngbot]

I see that you just added the pr: merge ready label, but the following checks are still failing:
     status "ci/circleci: api_golden_checks" is failing
     status "ci/circleci: build_release_packages" is failing
     status "ci/circleci: e2e_tests" is failing
     status "ci/circleci: ivy_test" is failing
     status "ci/circleci: lint" is failing
     status "ci/circleci: prerender_build" is failing
     status "ci/circleci: tests_browserstack" is failing
     status "ci/circleci: tests_local_browsers" is failing
     status "ci/circleci: tests_saucelabs" is failing
     status "ci/circleci: bazel_build" is pending

If you want your PR to be merged, it has to pass all the CI checks.

If you can't get the PR to a green state due to flakes or broken master, please try rebasing to master and/or restarting the CI job. If that fails and you believe that the issue is not due to your change, please contact the caretaker and ask for help.

[crisbeto]

Should we have a comment about why we're disabling it? Also do we need to do this on all components or just the one that's around the template content?

Finally, the setup between tabs and stepper is pretty similar so my guess is that we might have to do the same there.

[jelbourn]

Yeah, I suspect we'll be doing this for any component that consumes a template, in which case we would just disable the lint. I wanted to do that in a separate PR.

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