Parser: Make skipping HTML comments optional.

test/cctest/test-api.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 24840 matching lines @@
   CHECK(proxy->GetTarget()->SameValue(target));
   CHECK(proxy->GetHandler()->SameValue(handler));
 
   proxy->Revoke();
   CHECK(proxy->IsProxy());
   CHECK(!target->IsProxy());
   CHECK(proxy->IsRevoked());
   CHECK(proxy->GetTarget()->SameValue(target));
   CHECK(proxy->GetHandler()->IsNull());
 }
+
+TEST(HTMLCommentInJavascript) {
+  // Regression test for crbug.com/573887.
+  LocalContext context;
+  v8::HandleScope scope(CcTest::isolate());
+
+  // Inline scripts - i.e. the contents of <script> tags - need to obey HTML
+  // comments, but JS sources - i.e., everything else - shouldn't.
+  const char* source = "<!--\n var result = 2 + 2;\n-->\n result";
+
+  v8::ScriptOrigin inline_script(v8_str(""));
+
+  v8::ScriptOrigin source_script(
+      v8_str("http://some-resource.net/"), Local<v8::Integer>(),
+      Local<v8::Integer>(), Local<Boolean>(), Local<v8::Integer>(),
+      Local<Boolean>(), Local<Value>(), Local<Boolean>(),
+      v8::False(CcTest::isolate()));
+
+  // Case 1: An inline script.
+  {
+    v8::MaybeLocal<v8::Script> script =
+        v8::Script::Compile(context.local(), v8_str(source), &inline_script);
+    CHECK(!script.IsEmpty());
+    CHECK_EQ(4, script.ToLocalChecked()
+                    ->Run(context.local())
+                    .ToLocalChecked()
+                    ->Int32Value(context.local())
+                    .FromJust());
+  }
+
+  // Case 2: Js source file.
+  {
+    v8::MaybeLocal<v8::Script> script =
+        v8::Script::Compile(context.local(), v8_str(source), &source_script);
+    CHECK(script.IsEmpty());
+  }
+}