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

src/assembler.cc

Index: src/assembler.cc
===================================================================
--- src/assembler.cc	(revision 13419)
+++ src/assembler.cc	(working copy)
@@ -1522,6 +1522,13 @@
     gdbjit_lineinfo_->SetPosition(assembler_->pc_offset(), pos, false);
   }
 #endif
+  if (user_data_ != NULL) {

[danno, 2013/02/01 13:43:10]

Why conditionally call? It seems like you should _always_ call the logger to log
the event, if the handler chooses to ignore the event then this null check
should be inside the handler.

+    LOG_CODE_EVENT(this->assembler()->isolate(),
+                   CodeLinePosInfoAddEvent(user_data_,
+                                           assembler_->pc_offset(),
+                                           pos,
+                                           JitCodeEvent::POSITION));
+  }
 }
 
 
@@ -1534,6 +1541,13 @@
     gdbjit_lineinfo_->SetPosition(assembler_->pc_offset(), pos, true);
   }
 #endif
+  if (user_data_ != NULL) {

[danno, 2013/02/01 13:43:10]

Same comment as above (please always call LOG_CODE_EVENT)

+    LOG_CODE_EVENT(this->assembler()->isolate(),
+                   CodeLinePosInfoAddEvent(user_data_,
+                                           assembler_->pc_offset(),
+                                           pos,
+                                           JitCodeEvent::STATEMENT_POSITION));
+  }
 }