Report promise reject with no handler (behind a flag).

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 17548 matching lines @@
       "var e = {__proto__: new Error()} \n"
       "throw e;                         \n";
   v8::V8::AddMessageListener(RethrowBogusErrorStackTraceHandler);
   v8::V8::SetCaptureStackTraceForUncaughtExceptions(true);
   CompileRun(source);
   v8::V8::SetCaptureStackTraceForUncaughtExceptions(false);
   v8::V8::RemoveMessageListeners(RethrowBogusErrorStackTraceHandler);
 }
 
 
+static int promise_message_counter = 0;
+
+
+static void CheckPromiseInMessage(v8::Handle<v8::Message> message,
+                                  v8::Handle<v8::Value> data) {
+  CHECK(!message->GetPromise().IsEmpty());
+  v8::ScriptOrigin origin = message->GetScriptOrigin();
+  promise_message_counter++;
+  if (promise_message_counter == 1) {
+    v8::Handle<v8::Value> expected = CcTest::global()->Get(v8_str("p"));
+    CHECK(expected->Equals(message->GetPromise()));
+    CHECK(data->Equals(v8_str("p1")));
+    // The message location should point to the call site of reject.
+    CHECK(origin.ResourceName()->Equals(v8_str("test1")));
+    CHECK_EQ(6, message->GetLineNumber());
+  } else if (promise_message_counter == 2) {
+    v8::Handle<v8::Value> expected = CcTest::global()->Get(v8_str("q"));
+    CHECK(expected->Equals(message->GetPromise()));
+    CHECK(data->Equals(v8_str("q2")));
+    // There is no valid location when a promise is rejected by returning a
+    // rejected promise, since the actual rejection happens in internal code.
+    CHECK(origin.ResourceName()->IsUndefined());
+    CHECK_EQ(0, message->GetLineNumber());
+  } else {
+    CHECK_EQ(3, promise_message_counter);
+    v8::Handle<v8::Value> expected = CcTest::global()->Get(v8_str("r"));
+    CHECK(expected->Equals(message->GetPromise()));
+    CHECK(data->Equals(v8_str("r3")));
+    // The message location should point to the throw site.
+    CHECK(origin.ResourceName()->Equals(v8_str("test3")));
+    CHECK_EQ(3, message->GetLineNumber());
+  }
+}
+
+
+TEST(PromiseMessage) {
+  i::FLAG_report_promise_reject = true;
+  LocalContext env;
+  v8::Isolate* isolate = env->GetIsolate();
+  v8::HandleScope scope(isolate);
+
+  v8::V8::AddMessageListener(CheckPromiseInMessage);
+  promise_message_counter = 0;
+  v8::Handle<v8::Promise> promise;
+
+  // Create promise o.
+  CompileRun(
+      "var reject;            \n"
+      "var o = new Promise(   \n"
+      "  function(res, rej) { \n"
+      "    reject = rej;      \n"
+      "  }                    \n"
+      ");                     \n");
+  promise = v8::Handle<v8::Promise>::Cast(CcTest::global()->Get(v8_str("o")));
+  CHECK(!promise->HasRejectHandler());
+  CHECK_EQ(0, promise_message_counter);
+
+  // Add reject handler to o.
+  CompileRun("o.catch(function() {});");
+  CHECK(promise->HasRejectHandler());
+  CHECK_EQ(0, promise_message_counter);
+
+  // Reject o. We expect no message.
+  CompileRun("reject('o0')");
+  CHECK(promise->HasRejectHandler());
+  CHECK_EQ(0, promise_message_counter);
+
+  // Create Promise p, which is rejected by calling reject.
+  CompileRun(
+      "var p = new Promise(   \n"
+      "  function(res, rej) { \n"
+      "    reject = rej;      \n"
+      "  }                    \n"
+      ");                     \n"
+      "reject('p1');          \n //@ sourceURL=test1");
+  CHECK_EQ(1, promise_message_counter);
+  // Fetch promise from container. It doesn't have a reject handler.
+  promise = v8::Handle<v8::Promise>::Cast(CcTest::global()->Get(v8_str("p")));
+  CHECK(!promise->HasRejectHandler());
+
+  // Handle the rejection. Promise p should now have a reject handler.
+  // q is rejected by returning Promise.reject().
+  CompileRun(
+      "var q = p.catch(                  \n"
+      "   function() {                   \n"
+      "     return Promise.reject('q2'); \n"
+      "   }                              \n"
+      ");                                \n //@ sourceURL=test2");
+  CHECK_EQ(2, promise_message_counter);
+  CHECK(promise->HasRejectHandler());
+  // Fetch promise from container. It doesn't have a reject handler.
+  promise = v8::Handle<v8::Promise>::Cast(CcTest::global()->Get(v8_str("q")));
+
+  // Handle the rejection. Promise q should now have a reject handler.
+  // r is rejected by throwing.
+  CompileRun(
+      "var r = p.catch( \n"
+      "  function() {   \n"
+      "    throw 'r3';  \n"
+      "  }              \n"
+      ");               \n //@ sourceURL=test3");
+  CHECK_EQ(3, promise_message_counter);
+  promise = v8::Handle<v8::Promise>::Cast(CcTest::global()->Get(v8_str("r")));
+}
+
+
 void AnalyzeStackOfEvalWithSourceURL(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
   v8::HandleScope scope(args.GetIsolate());
   v8::Handle<v8::StackTrace> stackTrace = v8::StackTrace::CurrentStackTrace(
       args.GetIsolate(), 10, v8::StackTrace::kDetailed);
   CHECK_EQ(5, stackTrace->GetFrameCount());
   v8::Handle<v8::String> url = v8_str("eval_url");
   for (int i = 0; i < 3; i++) {
     v8::Handle<v8::String> name =
         stackTrace->GetFrame(i)->GetScriptNameOrSourceURL();
@@ skipping 5815 matching lines @@
       "  var foobXXXXX";  // Too many bytes which look like incomplete chars!
   char chunk2[] =
       "r = 13;\n"
       "  return foob\xeb\x91\x80\x80\x80r;\n"
       "}\n";
   for (int i = 0; i < 5; ++i) chunk1[strlen(chunk1) - 5 + i] = reference[i];
 
   const char* chunks[] = {chunk1, chunk2, "foo();", NULL};
   RunStreamingTest(chunks, v8::ScriptCompiler::StreamedSource::UTF8, false);
 }