Revert of [profiler] reduce incorrectly unaccounted ticks.

src/profiler/tick-sample.cc

 // Copyright 2013 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.
 
 #include "src/profiler/tick-sample.h"
 
 #include "include/v8-profiler.h"
 #include "src/counters.h"
 #include "src/frames-inl.h"
 #include "src/msan.h"
@@ skipping 207 matching lines @@
     i::Address* external_callback_entry_ptr =
         scope->callback_entrypoint_address();
     sample_info->external_callback_entry =
         external_callback_entry_ptr == nullptr ? nullptr
                                                : *external_callback_entry_ptr;
   }
 
   i::SafeStackFrameIterator it(isolate, reinterpret_cast<i::Address>(regs->fp),
                                reinterpret_cast<i::Address>(regs->sp),
                                js_entry_sp);
-  if (it.done()) return true;
+
+  // If at this point iterator does not see any frames,
+  // is usually means something is wrong with the FP,
+  // e.g. it is used as a general purpose register in the function.
+  // Bailout.
+  if (it.done()) return false;
 
   size_t i = 0;
   if (record_c_entry_frame == kIncludeCEntryFrame &&
       (it.top_frame_type() == internal::StackFrame::EXIT ||
        it.top_frame_type() == internal::StackFrame::BUILTIN_EXIT)) {
     frames[i++] = isolate->c_function();
   }
   i::RuntimeCallTimer* timer =
       isolate->counters()->runtime_call_stats()->current_timer();
   for (; !it.done() && i < frames_limit; it.Advance()) {
@@ skipping 28 matching lines @@
                       bool use_simulator_reg_state) {
   v8::TickSample::Init(reinterpret_cast<v8::Isolate*>(isolate), state,
                        record_c_entry_frame, update_stats,
                        use_simulator_reg_state);
   if (pc == nullptr) return;
   timestamp = base::TimeTicks::HighResolutionNow();
 }
 
 }  // namespace internal
 }  // namespace v8