fix(bazel): ng_package(data) should support non-text files

[jbedard]

Today non-text ng_package(data) files might (always?) get corrupted because they get read+written as utf8, for example font or image files.

I think the utf8 will be used by default for non-binary files, so the param is not needed at all?

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• angular.io application / infrastructure changes
• Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

The bazel ng_package(data) now supports non-utf8 files.

Does this PR introduce a breaking change?

• Yes
• No

Other information

[alexeagle]

awesome, nice job finding a way to test it

[AndrewKushnir]

@jbedard thanks for the PR! Could you please have a look at the lint error here:

https://circleci.com/gh/angular/angular/459499

Thank you.

[AndrewKushnir]

@alexeagle should this PR go into master only or master and patch?

[jbedard]

Fixed 🤞

Would be great if this was in a patch release...

[AndrewKushnir]

Thanks @jbedard and @alexeagle, this PR is now merged into master and patch branches.

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