Log grammar collisions without a stack trace.

[kittaakos]

Signed-off-by: Akos Kitta kittaakos@typefox.io

What it does

From now on, TextMate grammar collisions are logged without a stack trace.

Before:

logger-protocol.ts:112 root WARN Error: 'cpp' language is remapped from 'source.cpp.embedded.macro' to 'source.cpp' scope
    at TextmateRegistry.push.../../packages/monaco/lib/browser/textmate/textmate-registry.js.TextmateRegistry.mapLanguageIdToTextmateGrammar (file:///Users/akos.kitta/git/theia/examples/electron/lib/43.bundle.js:2259:26)
    at file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7275:125
    at pushContribution (file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7133:32)
    at _loop_15 (file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7275:21)
    at file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7284:25
logger-protocol.ts:112 root WARN Error: a registered grammar configuration for 'cpp' language is overridden
    at TextmateRegistry.push.../../packages/monaco/lib/browser/textmate/textmate-registry.js.TextmateRegistry.registerGrammarConfiguration (file:///Users/akos.kitta/git/theia/examples/electron/lib/43.bundle.js:2297:26)
    at file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7276:133
    at pushContribution (file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7133:32)
    at _loop_15 (file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7276:21)
    at file:///Users/akos.kitta/git/theia/examples/electron/lib/53.bundle.js:7284:25

After:

logger-protocol.ts:112 root WARN 'cpp' language is remapped from 'source.cpp.embedded.macro' to 'source.cpp' scope
logger-protocol.ts:112 root WARN a registered grammar configuration for 'cpp' language is overridden

How to test

Start the app and check the browser logs.

Review checklist

• as an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• as a reviewer, I agree to behave in accordance with the review guidelines

[akosyakov]

I think we can do it always.

[kittaakos]

I think we can do it always.

Even better, let me do an update. Do you want me to keep this customizable? Or dropping the Error and logging the message (string) is sufficient?

[akosyakov]

lgtm