console.timeEnd shouldn't throw if there is no matching label

[DavidBruant]

There is a test expecting that behavior, but it probably isn't a sensible behavior.

A better behavior would be to fail silently while printing a message to stderr saying there is no corresponding label.

[cjihrig]

I'm not saying the current behavior is write or wrong, but it looks like it's been that way for quite a while.

Seems that Chrome does not throw in a similar situation.

[DavidBruant]

Seems that Chrome does not throw in a similar situation.

Neither does Firefox AFAICT

[mscdex]

If we're going to use Chrome and Firefox as models, then we should just ignore invalid labels and not print anything.

[cjihrig]

@bnoordhuis do you remember the original purpose for throwing (it looks like you landed the commit)?

We would still have to assume that someone, somewhere is relying on that exception being thrown, and treat this as a semver major change

[jasnell]

Fixed in #5901