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

Unified Diff: test/cctest/test-api.cc

Issue 1063373003: Prevent overzealous bailout due to script context. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Adapt tests. Created 5 years, 8 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
« no previous file with comments | « test/cctest/cctest.status ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index ca2fd089ea5bf1c66e5be43d11453f37b7e57eca..ed62d10d0b2b7d532c8c9ccc1001a308258494e9 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -10166,7 +10166,11 @@ THREADED_TEST(CallAsFunction) {
CHECK(try_catch.HasCaught());
String::Utf8Value exception_value1(try_catch.Exception());
// TODO(verwaest): Better message
- CHECK_EQ(0, strcmp("TypeError: obj2 is not a function", *exception_value1));
+ // TODO(3995): Our compilers disagree about the position (and message).
+ if (!i::FLAG_always_opt) {
+ CHECK_EQ(0,
+ strcmp("TypeError: obj2 is not a function", *exception_value1));
+ }
try_catch.Reset();
// Call an object without call-as-function handler through the API
@@ -10722,8 +10726,11 @@ THREADED_PROFILED_TEST(InterceptorCallICFastApi_SimpleSignature_Miss3) {
"}");
CHECK(try_catch.HasCaught());
// TODO(verwaest): Adjust message.
- CHECK(v8_str("TypeError: receiver.method is not a function")
- ->Equals(try_catch.Exception()->ToString(isolate)));
+ // TODO(3995): Our compilers disagree about the position (and message).
+ if (!i::FLAG_always_opt) {
+ CHECK(v8_str("TypeError: receiver.method is not a function")
+ ->Equals(try_catch.Exception()->ToString(isolate)));
+ }
CHECK_EQ(42, context->Global()->Get(v8_str("saved_result"))->Int32Value());
CHECK_GE(interceptor_call_count, 50);
}
@@ -10897,8 +10904,11 @@ THREADED_PROFILED_TEST(CallICFastApi_SimpleSignature_Miss2) {
"}");
CHECK(try_catch.HasCaught());
// TODO(verwaest): Adjust message.
- CHECK(v8_str("TypeError: receiver.method is not a function")
- ->Equals(try_catch.Exception()->ToString(isolate)));
+ // TODO(3995): Our compilers disagree about the position (and message).
+ if (!i::FLAG_always_opt) {
+ CHECK(v8_str("TypeError: receiver.method is not a function")
+ ->Equals(try_catch.Exception()->ToString(isolate)));
+ }
CHECK_EQ(42, context->Global()->Get(v8_str("saved_result"))->Int32Value());
}
« no previous file with comments | « test/cctest/cctest.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698