Correctly annotate eval origin.

src/frames.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_FRAMES_H_
 #define V8_FRAMES_H_
 
 #include "src/allocation.h"
 #include "src/handles.h"
 #include "src/safepoint-table.h"
@@ skipping 683 matching lines @@
   static inline bool IsConstructFrame(Address fp);
 
   // Used by OptimizedFrames and StubFrames.
   void IterateCompiledFrame(ObjectVisitor* v) const;
 
  private:
   friend class StackFrame;
   friend class SafeStackFrameIterator;
 };
 
+class JavaScriptFrame;
 
 class FrameSummary BASE_EMBEDDED {
  public:
   FrameSummary(Object* receiver, JSFunction* function,
                AbstractCode* abstract_code, int code_offset,
                bool is_constructor);
 
+  static FrameSummary GetFirst(JavaScriptFrame* frame);
+
   Handle<Object> receiver() { return receiver_; }
   Handle<JSFunction> function() { return function_; }
   Handle<AbstractCode> abstract_code() { return abstract_code_; }
   int code_offset() { return code_offset_; }
   bool is_constructor() { return is_constructor_; }
 
   void Print();
 
  private:
   Handle<Object> receiver_;
@@ skipping 165 matching lines @@
       int* data, HandlerTable::CatchPrediction* prediction) override;
 
   // Returns the current offset into the bytecode stream.
   int GetBytecodeOffset() const;
 
   // Updates the current offset into the bytecode stream, mainly used for stack
   // unwinding to continue execution at a different bytecode offset.
   void PatchBytecodeOffset(int new_offset);
 
   // Returns the frame's current bytecode array.
-  Object* GetBytecodeArray() const;
+  BytecodeArray* GetBytecodeArray() const;
 
   // Updates the frame's BytecodeArray with |bytecode_array|. Used by the
   // debugger to swap execution onto a BytecodeArray patched with breakpoints.
   void PatchBytecodeArray(Object* bytecode_array);

[Michael Starzinger, 2016/04/05 09:56:49]

nit: Let's also change the signature of the setter to only allow
"BytecodeArray". If the getter guarantees proper type, then the setter needs to
enforce that guarantee.

 
   // Access to the interpreter register file for this frame.
   Object* GetInterpreterRegister(int register_index) const;
 
   // Build a list with summaries for this frame including all inlined frames.
   void Summarize(List<FrameSummary>* frames) override;
 
  protected:
   inline explicit InterpretedFrame(StackFrameIteratorBase* iterator);
 
@@ skipping 289 matching lines @@
 
 
 // Reads all frames on the current stack and copies them into the current
 // zone memory.
 Vector<StackFrame*> CreateStackMap(Isolate* isolate, Zone* zone);
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_FRAMES_H_