[d8] console methods must not throw.

src/builtins/builtins-console.cc

Index: src/builtins/builtins-console.cc
diff --git a/src/builtins/builtins-console.cc b/src/builtins/builtins-console.cc
index a021c8a75b2e4d90dfee99da7daf0a3bde4fc7dd..a43fe136d0a802c90950da915f33fc5be042ca78 100644
--- a/src/builtins/builtins-console.cc
+++ b/src/builtins/builtins-console.cc
@@ -45,6 +45,8 @@ namespace internal {
     if (isolate->console_delegate()) {            \
       debug::ConsoleCallArguments wrapper(args);  \
       isolate->console_delegate()->name(wrapper); \
+      CHECK(!isolate->has_pending_exception());   \

[kozy, 2017/04/26 23:00:42]

What is motivation behind this check?
I just wondering because there is some current console-spec work in progress and
one of the proposals is to use real toString [1] for
console.time({toString:()=>"123"}) and in some other methods.

[1] crbug.com/696805

+      CHECK(!isolate->has_scheduled_exception()); \
     }                                             \
     return isolate->heap()->undefined_value();    \
   }