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

src/lithium.cc

 // 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 398 matching lines @@
     info->set_bailout_reason("not enough virtual registers (regalloc)");
     return NULL;
   }
 
   return chunk;
 }
 
 
 Handle<Code> LChunk::Codegen(Code::Kind kind) {
   MacroAssembler assembler(info()->isolate(), NULL, 0);
+  if (info()->isolate()->logger()->is_code_event_handler_enabled()) {

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

Other Logger APIs do "is_enabled" checking inside of the API implementation so
that the client doesn't have to do it. Please remove this check, and always call
 " info()->isolate()->logger()->CodeStartLinePosInfoRecordEvent()" here and
below (CodeEndLinePosInfoRecordEvent). Add a "if
(!info()->isolate()->logger()->is_code_event_handler_enabled()) return" inside
of the API implementation.

+    void* line_info =

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

This should be jit_handler_data

+        info()->isolate()->logger()->CodeStartLinePosInfoRecordEvent();
+    assembler.positions_recorder()->InitializeUserData(line_info);

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

InitializeUserData should be InitializeJitHandlerData. Furthermore, I think you
should pass in the position recorder to CodeStartLinePosInfoRecordEvent which
should call InitializeJitHandlerData, don't do it here.

+  }
   LCodeGen generator(this, &assembler, info());
 
   MarkEmptyBlocks();
 
   if (generator.GenerateCode()) {
     if (FLAG_trace_codegen) {
       PrintF("Crankshaft Compiler - ");
     }
     CodeGenerator::MakeCodePrologue(info());
     Code::Flags flags = Code::ComputeFlags(kind);
     Handle<Code> code =
         CodeGenerator::MakeCodeEpilogue(&assembler, flags, info());
     generator.FinishCode(code);
+
+    if (info()->isolate()->logger()->is_code_event_handler_enabled()) {
+      if (!code.is_null()) {
+        void* line_info =
+            assembler.positions_recorder()->DetachJITLineInfo();
+        LOG_CODE_EVENT(info()->isolate(),
+                       CodeEndLinePosInfoRecordEvent(*code, line_info));

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

line_info -> jit_handler_data

+      }
+    }
     CodeGenerator::PrintCode(code, info());
     return code;
   }
   return Handle<Code>::null();
 }
 
 
 } }  // namespace v8::internal