Embedded javascript string with `//` have unexpected behavrio

[alexr00]

Original from @namorzyny in microsoft/vscode#105779

Steps to Reproduce:

1. Input some code like: <input type="button" value="text" onclick="location.href='https://www.google.com'">
2. The code will display as a comment:

This only occurs if the // is in the string. The character before the // will get invalid.illegal.newline.js.

[alexr00]

This issue doesn't occur with just the javascript grammar, only when its embedded like this.

[alexr00]

Thanks to some investigation from @mjbvz, the issue appears to be here:

html.tmbundle/Syntaxes/HTML.plist

Lines 341 to 359 in 0c3d5ee

	<string>string.quoted.double.html</string>
	<key>patterns</key>
	<array>
	<dict>
	<key>captures</key>
	<dict>
	<key>0</key>
	<dict>
	<key>patterns</key>
	<array>
	<dict>
	<key>include</key>
	<string>source.js</string>
	</dict>
	</array>
	</dict>
	</dict>
	<key>match</key>
	<string>([^\n"/]|/(?![/*]))+</string>

Original comment is microsoft/vscode#112189 (comment)

[rahul3v]

Well, same will happen when you use /* as well