feat(@ngtools/webpack): replace server bootstrap code

[FrozenPandaz]

Currently the loader is only replacing the bootstrap code of platformBrowser.

It should replace the bootstrap code of platformServer as well.

I also made it expandable to other platforms. We should probably add webworker platform as well.

I can do that in a separate pull request?

Motivation for this is better support of platform-server via @ngtools/webpack

[FrozenPandaz]

cc @hansl @filipesilva

[filipesilva]

@FrozenPandaz can you add a test that shows these changes work?

A good place to add would be side by side with the other webpack plugin tests in https://github.com/angular/angular-cli/tree/master/tests/e2e/tests/packages/webpack

These tests will create a new project from the files in https://github.com/angular/angular-cli/tree/master/tests/e2e/assets/webpack.

You'll need a new project in the last folder, and a new file that tests it in the first.

[FrozenPandaz]

@filipesilva yes i will add tests, thanks for pointing me to the right place i was actually confused about that because theres a loader spec which looks quite empty.

[googlebot]

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

[googlebot]

CLAs look good, thanks!

[FrozenPandaz]

@filipesilva I've added tests. Please take a look.

[filipesilva]

Good job on the test setup! I'm requesting Hans's review on this as the authority on the plugin.

[FrozenPandaz]

I rebased this again. @hansl

[demurgos]

Could you provide some updates about this issue ?

[Reuvenc]

Anything new regarding this PR? It is extremely important for our project.

[googlebot]

CLAs look good, thanks!

[FrozenPandaz]

@hansl This should be ready for you to take a look

[hansl]

LGTM, thanks!

[MarkPieszak]

Saw it merged to master, any ideas when we're releasing to npm?
@hansl

[paulogr]

Is this depends on #5547?

[FrozenPandaz]

Nope this does not depend on #5547

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