Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2053)

Unified Diff: test/cctest/test-code-stub-assembler.cc

Issue 2650803003: [inspector] change target promise for kDebugWillHandle & kDebugDidHandle (Closed)
Patch Set: added missing guard in asyncTaskCreated Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/cctest/test-code-stub-assembler.cc
diff --git a/test/cctest/test-code-stub-assembler.cc b/test/cctest/test-code-stub-assembler.cc
index 61c6b7bf7ed33c673e20b6945bc21e06e67700ec..019ce1bd503bdd806116ff70554d42cb891dc7b5 100644
--- a/test/cctest/test-code-stub-assembler.cc
+++ b/test/cctest/test-code-stub-assembler.cc
@@ -1749,7 +1749,7 @@ TEST(IsPromiseHookEnabled) {
CodeAssemblerTester data(isolate, kNumParams);
CodeStubAssembler m(data.state());
- m.Return(m.SelectBooleanConstant(m.IsPromiseHookEnabled()));
+ m.Return(m.SelectBooleanConstant(m.IsPromiseHookEnabledOrDebugIsActive()));
Handle<Code> code = data.GenerateCode();
CHECK(!code.is_null());
@@ -1847,7 +1847,6 @@ TEST(AllocatePromiseReactionJobInfo) {
CHECK(promise_info->deferred_on_resolve()->IsUndefined(isolate));
CHECK(promise_info->deferred_on_reject()->IsUndefined(isolate));
CHECK(promise_info->context()->IsContext());
- CHECK_EQ(kDebugPromiseNoID, promise_info->debug_id());
}
TEST(AllocatePromiseResolveThenableJobInfo) {
@@ -1886,7 +1885,6 @@ TEST(AllocatePromiseResolveThenableJobInfo) {
CHECK(promise_info->then()->IsJSFunction());
CHECK(promise_info->resolve()->IsJSFunction());
CHECK(promise_info->reject()->IsJSFunction());
- CHECK_EQ(kDebugPromiseNoID, promise_info->debug_id());
CHECK(promise_info->context()->IsContext());
}

Powered by Google App Engine
This is Rietveld 408576698