This patch is the propagation version of https://codereview.chromium.org/10824032 patch

include/v8.h

 // 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 3003 matching lines @@
  * \param function the address of the function that's being entered.
  * \param return_addr_location points to a location on stack where the machine
  *    return address resides. This can be used to identify the caller of
  *    \p function, and/or modified to divert execution when \p function exits.
  *
  * \note the entry hook must not cause garbage collection.
  */
 typedef void (*FunctionEntryHook)(uintptr_t function,
                                   uintptr_t return_addr_location);
 
-

[danno, 2012/12/20 16:27:20]

nit: Please don't remove whitespace not in accordance with our coding style,
which is 2 lines between declarations/functions/etc.

 /**
  * A JIT code event is issued each time code is added, moved or removed.
  *
  * \note removal events are not currently issued.
  */
 struct JitCodeEvent {
   enum EventType {
     CODE_ADDED,
     CODE_MOVED,
-    CODE_REMOVED
+    CODE_REMOVED,
+    CODE_ADD_LINE_POS_INFO,
+    CODE_START_LINE_INFO_RECORDING,
+    CODE_END_LINE_INFO_RECORDING
   };
 
   // Type of event.
   EventType type;
   // Start of the instructions.
   void* code_start;
   // Size of the instructions.
   size_t code_len;
+  // It can be Script info for CODE_ADDDED event or line_info_struct

[danno, 2012/12/20 16:27:20]

How about removing the "It can be", and only two DDs in ADDED

+  // for CODE_ADD_LINE_POS_INFO and CODE_END_LINE_INFO_RECORDING event
+  void* userdata;

[danno, 2012/12/20 16:27:20]

user_data is probably better. Also, why not have two fields that are strongly
typed for CODE_ADDED and CODE_ADD_LINE_POS_INFO/CODE_END_LINE_INFO_RECORDING
rather than this single void*?

 
   union {
     // Only valid for CODE_ADDED.
     struct {
       // Name of the object associated with the code, note that the string is
       // not zero-terminated.
       const char* str;
       // Number of chars in str.
       size_t len;
     } name;
+
+    // Only valid for CODE_ADD_LINE_INFO

[danno, 2012/12/20 16:27:20]

CODE_ADD_LINE_POS_INFO

+    struct {
+      // PC offset
+      unsigned offset;

[danno, 2012/12/20 16:27:20]

Why use unsigned rather than a size_t? This seems more consistent with the rest
of the API.

+      // Code postion
+      unsigned pos;

[danno, 2012/12/20 16:27:20]

Same here

+      // Whether this position is statement

[danno, 2012/12/20 16:27:20]

How about you clarify what it is if it isn't a statement, e.g. "Indicates
whether this position is a statement (as opposed to a XXXX)" 

+      bool is_statement;
+    } line_info;
+
     // New location of instructions. Only valid for CODE_MOVED.
     void* new_code_start;
   };
 };
 
 /**
  * Option flags passed to the SetJitCodeEventHandler function.
  */
 enum JitCodeEventOptions {
   kJitCodeEventDefault = 0,
@@ skipping 1800 matching lines @@
 
 
 }  // namespace v8
 
 
 #undef V8EXPORT
 #undef TYPE_CHECK
 
 
 #endif  // V8_H_