etw: add event messages

[joaocgreis]

Pull Request check-list

Please make sure to review and check all of these items:

• Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with
  this change (including linting)?
• Is the commit message formatted according to [CONTRIBUTING.md][0]?

Affected core subsystem(s)

ETW

Description of change

Note: #5657 should land before this, because this does not include the generated files.

A log of ETW events can be produced with:

logman start NodejsDCS -p NodeJS-ETW-provider -o node_log.etl -ets
:: Do something with node.exe
logman stop NodejsDCS -ets

The resulting etl file can be open in Event Viewer.

Without this change, the description box shows an error message:

The description for Event ID 23 from source NodeJS-ETW-provider cannot be found. Either the component that raises this event is not installed on your local computer or the installation is corrupted. You can install or repair the component on the local computer.

If the event originated on another computer, the display information had to be saved with the event.

The following information was included with the event: 


The message id for the desired message could not be found

This adds descriptions to all events, replacing the error message in the Event Viewer.

cc @nodejs/platform-windows

[bnoordhuis]

LGTM. Just a suggestion: "Method Load" is kind of a bad name, "Function Compiled" is closer to the truth.

[joaocgreis]

CI: https://ci.nodejs.org/job/node-test-pull-request/2110/ all ok except a slave failure

Landed in: 1845c4f

Thanks @bnoordhuis !

[benjamingr]

Thanks for the fix - I tested this locally and I like the change, forgot to comment.

[jasnell]

is this relevant to v4 at all?

[joaocgreis]

@jasnell I don't think so, this is a new feature.

Added the don't land label, feel free to remove if that was not the correct thing or not consensual.

[jasnell]

Works for me.