Change inlined stack frame iterator to use code and pc instead of frame.

runtime/vm/stack_frame.cc

Index: runtime/vm/stack_frame.cc
===================================================================
--- runtime/vm/stack_frame.cc	(revision 27137)
+++ runtime/vm/stack_frame.cc	(working copy)
@@ -343,25 +343,22 @@
 }
 
 
-InlinedFunctionsIterator::InlinedFunctionsIterator(StackFrame* frame)
+InlinedFunctionsIterator::InlinedFunctionsIterator(const Code& code, uword pc)
   : index_(0),
-    code_(Code::Handle()),
+    code_(Code::Handle(code.raw())),
     deopt_info_(DeoptInfo::Handle()),
     function_(Function::Handle()),
-    pc_(0),
+    pc_(pc),
     deopt_instructions_(),
     object_table_(Array::Handle()) {

[siva, 2013/09/04 19:32:42]

Need an assertion here to ensure that the passed in 'code' contains 'pc'.

[srdjan, 2013/09/04 19:36:03]

Done.

-  ASSERT(frame != NULL);
-  code_ = frame->LookupDartCode();
   ASSERT(code_.is_optimized());
   intptr_t deopt_reason = kDeoptUnknown;
-  deopt_info_ = code_.GetDeoptInfoAtPc(frame->pc(), &deopt_reason);
+  deopt_info_ = code_.GetDeoptInfoAtPc(pc, &deopt_reason);
   if (deopt_info_.IsNull()) {
     // This is the case when a call without deopt info in optimized code
     // throws an exception. (e.g. in the parameter copying prologue).
     // In that case there won't be any inlined frames.
     function_ = code_.function();
-    pc_ = frame->pc();
     ASSERT(pc_ != 0);
   } else {
     // Unpack deopt info into instructions (translate away suffixes).