Add the ability to throw an error to python debugger

This patch makes it possible to throw an error from the python debugger client using the `throw` command.

JerryScript-DCO-1.0-Signed-off-by: Daniel Balla dballa@inf.u-szeged.hu

Author: Daniel Balla

jerry-core/debugger/debugger.c

@@ -142,14 +142,15 @@ jerry_debugger_send_backtrace (uint8_t *recv_buffer_p) /**< pointer the the rece
 } /* jerry_debugger_send_backtrace */
 
 /**
- * Send result of evaluated expression.
+ * Send result of evaluated expression or throw an error.
  *
  * @return true - if no error is occured
  *         false - otherwise
  */
 static bool
-jerry_debugger_send_eval (const lit_utf8_byte_t *eval_string_p, /**< evaluated string */
-                          size_t eval_string_size) /**< evaluated string size */
+jerry_debugger_send_eval_or_throw (const lit_utf8_byte_t *eval_string_p, /**< evaluated string */
+                                   size_t eval_string_size, /**< evaluated string size */
+                                   bool *is_eval) /**< throw or eval call*/
 {
   JERRY_ASSERT (JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_CONNECTED);
   JERRY_ASSERT (!(JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_VM_IGNORE));
@@ -160,11 +161,21 @@ jerry_debugger_send_eval (const lit_utf8_byte_t *eval_string_p, /**< evaluated s
 
   if (!ECMA_IS_VALUE_ERROR (result))
   {
+    if (!*is_eval)
+    {
+      JERRY_DEBUGGER_SET_FLAGS (JERRY_DEBUGGER_THROW_ERROR_FLAG);
+      JERRY_CONTEXT (error_value) = result;
+      JERRY_DEBUGGER_CLEAR_FLAGS (JERRY_DEBUGGER_VM_STOP);
+      JERRY_CONTEXT (debugger_stop_context) = NULL;
+      return true;
+    }
+
     ecma_value_t to_string_value = ecma_op_to_string (result);
     ecma_free_value (result);
     result = to_string_value;
   }
 
+  *is_eval = true;
   ecma_value_t message = result;
   uint8_t type = JERRY_DEBUGGER_EVAL_OK;
 
@@ -215,7 +226,7 @@ jerry_debugger_send_eval (const lit_utf8_byte_t *eval_string_p, /**< evaluated s
   ecma_free_value (message);
 
   return success;
-} /* jerry_debugger_send_eval */
+} /* jerry_debugger_send_eval_or_throw */
 
 /**
  * Suspend execution for a given time.
@@ -272,6 +283,7 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
   if (*expected_message_type_p != 0)
   {
     JERRY_ASSERT (*expected_message_type_p == JERRY_DEBUGGER_EVAL_PART
+                  || *expected_message_type_p == JERRY_DEBUGGER_THROW_PART
                   || *expected_message_type_p == JERRY_DEBUGGER_CLIENT_SOURCE_PART);
 
     jerry_debugger_uint8_data_t *uint8_data_p = (jerry_debugger_uint8_data_t *) *message_data_p;
@@ -318,9 +330,15 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
     }
 
     bool result = false;
+    bool is_eval = true;
     if (*expected_message_type_p == JERRY_DEBUGGER_EVAL_PART)
     {
-      result = jerry_debugger_send_eval (string_p, uint8_data_p->uint8_size);
+      result = jerry_debugger_send_eval_or_throw (string_p, uint8_data_p->uint8_size, &is_eval);
+    }
+    else if (*expected_message_type_p == JERRY_DEBUGGER_THROW_PART)
+    {
+      is_eval = false;
+      result = jerry_debugger_send_eval_or_throw (string_p, uint8_data_p->uint8_size, &is_eval);
     }
     else
     {
@@ -504,6 +522,7 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
     }
 
     case JERRY_DEBUGGER_EVAL:
+    case JERRY_DEBUGGER_THROW:
     {
       if (message_size < sizeof (jerry_debugger_receive_eval_first_t) + 1)
       {
@@ -516,6 +535,7 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
 
       uint32_t eval_size;
       memcpy (&eval_size, eval_first_p->eval_size, sizeof (uint32_t));
+      bool is_eval = (recv_buffer_p[0] == JERRY_DEBUGGER_EVAL);
 
       if (eval_size <= JERRY_DEBUGGER_MAX_RECEIVE_SIZE - sizeof (jerry_debugger_receive_eval_first_t))
       {
@@ -525,8 +545,13 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
           jerry_debugger_close_connection ();
           return false;
         }
+        bool ret_val = jerry_debugger_send_eval_or_throw ((lit_utf8_byte_t *) (eval_first_p + 1), eval_size, &is_eval);
+        if (!is_eval)
+        {
+          *resume_exec_p = true;
+        }
 
-        return jerry_debugger_send_eval ((lit_utf8_byte_t *) (eval_first_p + 1), eval_size);
+        return ret_val;
       }
 
       jerry_debugger_uint8_data_t *eval_uint8_data_p;
@@ -543,7 +568,9 @@ jerry_debugger_process_message (uint8_t *recv_buffer_p, /**< pointer the the rec
               message_size - sizeof (jerry_debugger_receive_eval_first_t));
 
       *message_data_p = eval_uint8_data_p;
-      *expected_message_type_p = JERRY_DEBUGGER_EVAL_PART;
+      *expected_message_type_p = is_eval ? JERRY_DEBUGGER_EVAL_PART
+                                         : JERRY_DEBUGGER_THROW_PART;
+
       return true;
     }
 

jerry-core/debugger/debugger.h

@@ -98,6 +98,7 @@ typedef enum
   JERRY_DEBUGGER_CLIENT_SOURCE_MODE = 1u << 7, /**< debugger waiting for client code */
   JERRY_DEBUGGER_CLIENT_NO_SOURCE = 1u << 8, /**< debugger leaving the client source loop */
   JERRY_DEBUGGER_CONTEXT_RESET_MODE = 1u << 9, /**< debugger and engine reinitialization mode */
+  JERRY_DEBUGGER_THROW_ERROR_FLAG = 1u << 10, /**< debugger client sent an error throw */
 } jerry_debugger_flags_t;
 
 /**
@@ -185,6 +186,8 @@ typedef enum
   JERRY_DEBUGGER_EVAL_PART = 17, /**< next message of evaluating a string */
 
   JERRY_DEBUGGER_MESSAGES_IN_MAX_COUNT, /**< number of different type of input messages */
+  JERRY_DEBUGGER_THROW = 18, /**< first message of the throw string */
+  JERRY_DEBUGGER_THROW_PART = 19, /**< next part of the throw message */
 } jerry_debugger_header_type_t;
 
 /**

jerry-core/vm/vm.c

@@ -2517,6 +2517,11 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
           frame_ctx_p->byte_code_p = byte_code_start_p;
 
           jerry_debugger_breakpoint_hit (JERRY_DEBUGGER_BREAKPOINT_HIT);
+          if (JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_THROW_ERROR_FLAG)
+          {
+            result = JERRY_CONTEXT (error_value);
+            goto error;
+          }
 #endif /* JERRY_DEBUGGER */
           continue;
         }
@@ -2539,6 +2544,11 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
                   || JERRY_CONTEXT (debugger_stop_context) == JERRY_CONTEXT (vm_top_context_p)))
           {
             jerry_debugger_breakpoint_hit (JERRY_DEBUGGER_BREAKPOINT_HIT);
+            if (JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_THROW_ERROR_FLAG)
+            {
+              result = JERRY_CONTEXT (error_value);
+              goto error;
+            }
             continue;
           }
 
@@ -2560,6 +2570,11 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
                   || JERRY_CONTEXT (debugger_stop_context) == JERRY_CONTEXT (vm_top_context_p)))
           {
             jerry_debugger_breakpoint_hit (JERRY_DEBUGGER_BREAKPOINT_HIT);
+            if (JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_THROW_ERROR_FLAG)
+            {
+              result = JERRY_CONTEXT (error_value);
+              goto error;
+            }
           }
 #endif /* JERRY_DEBUGGER */
           continue;
@@ -2699,9 +2714,19 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
           && !(frame_ctx_p->bytecode_header_p->status_flags & CBC_CODE_FLAGS_DEBUGGER_IGNORE)
           && !(JERRY_CONTEXT (debugger_flags) & (JERRY_DEBUGGER_VM_IGNORE_EXCEPTION | JERRY_DEBUGGER_VM_IGNORE)))
       {
+        /* Save the error to a local value, because the engine enters breakpoint mode after,
+           therefore an evaluation error, or user-created error throw would overwrite it. */
+        ecma_value_t local_err = JERRY_CONTEXT (error_value);
         if (jerry_debugger_send_exception_string ())
         {
           jerry_debugger_breakpoint_hit (JERRY_DEBUGGER_EXCEPTION_HIT);
+          if (JERRY_CONTEXT (debugger_flags) & JERRY_DEBUGGER_THROW_ERROR_FLAG)
+          {
+            result = JERRY_CONTEXT (error_value);
+            ecma_free_value (local_err);
+            goto error;
+          }
+          JERRY_CONTEXT (error_value) = local_err;
         }
       }
 #endif /* JERRY_DEBUGGER */

jerry-debugger/jerry-client-ws.py

@@ -87,6 +87,8 @@
 JERRY_DEBUGGER_GET_BACKTRACE = 15
 JERRY_DEBUGGER_EVAL = 16
 JERRY_DEBUGGER_EVAL_PART = 17
+JERRY_DEBUGGER_THROW = 18
+JERRY_DEBUGGER_THROW_PART = 19
 
 MAX_BUFFER_SIZE = 128
 WEBSOCKET_BINARY_FRAME = 2
@@ -435,6 +437,8 @@ def send_string(self, args, message_type):
 
         if message_type == JERRY_DEBUGGER_EVAL:
             message_type = JERRY_DEBUGGER_EVAL_PART
+        elif message_type == JERRY_DEBUGGER_THROW:
+            message_type = JERRY_DEBUGGER_THROW_PART
         else:
             message_type = JERRY_DEBUGGER_CLIENT_SOURCE_PART
 
@@ -463,6 +467,10 @@ def do_eval(self, args):
 
     do_e = do_eval
 
+    def do_throw(self, args):
+        """ Throw an exception """
+        self.send_string(args, JERRY_DEBUGGER_THROW)
+
     def do_exception(self, args):
         """ Config the exception handler module """
         if not args:

tests/debugger/do_help.expected

@@ -4,7 +4,7 @@ Stopped at tests/debugger/do_help.js:15
 
 Documented commands (type help <topic>):
 ========================================
-b          bt        delete   e          help      ms    quit    source
+b          bt        delete   e          help      ms    quit    source  throw
 backtrace  c         display  eval       list      n     s       src   
 break      continue  dump     exception  memstats  next  scroll  step  
 

tests/debugger/do_throw.cmd

@@ -0,0 +1 @@
+throw new Error('Once upon a time there lived in a certain village a little country girl, the prettiest creature who was ever seen. Her mother was excessively fond of her; and her grandmother doted on her still more. This good woman had a little red riding hood made for her. It suited the girl so extremely well that everybody called her Little Red Riding Hood. One day her mother, having made some cakes, said to her, "Go, my dear, and see how your grandmother is doing, for I hear she has been very ill. Take her a cake, and this little pot of butter."')

tests/debugger/do_throw.expected

@@ -0,0 +1,3 @@
+Connecting to: localhost:5001
+Stopped at tests/debugger/do_throw.js:19
+(jerry-debugger) throw new Error('Once upon a time there lived in a certain village a little country girl, the prettiest creature who was ever seen. Her mother was excessively fond of her; and her grandmother doted on her still more. This good woman had a little red riding hood made for her. It suited the girl so extremely well that everybody called her Little Red Riding Hood. One day her mother, having made some cakes, said to her, "Go, my dear, and see how your grandmother is doing, for I hear she has been very ill. Take her a cake, and this little pot of butter."')

tests/debugger/do_throw.js

@@ -0,0 +1,19 @@
+// Copyright JS Foundation and other contributors, http://js.foundation
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+function inevitable() {
+  print("An error will be thrown.");
+}
+
+inevitable();