Revert "Reland "Don't use instrumentation stubs for native methods in debuggable""

This reverts commit 6fb0acc14459a856c35b642e3368aff853259260.

Reason for revert: Breaks android.jvmti.cts.JvmtiHostTest
https://buganizer.corp.google.com/issues/237991413

Change-Id: I00fb58080693ddebc03c7b62ea67c91150ef7a21

1 files changed, 3 insertions, 4 deletions

diff --git a/runtime/quick_exception_handler.cc b/runtime/quick_exception_handler.cc
index 40a1c16905..8adc3b3e49 100644
--- a/runtime/quick_exception_handler.cc
+++ b/runtime/quick_exception_handler.cc
@@ -399,10 +399,9 @@ class DeoptimizeStackVisitor final : public StackVisitor {
       return true;
     } else if (method->IsNative()) {
       // If we return from JNI with a pending exception and want to deoptimize, we need to skip
-      // the native method. The top method is a runtime method, the native method comes next.
-      // We also deoptimize due to method instrumentation reasons from method entry / exit
-      // callbacks. In these cases native method is at the top of stack.
-      CHECK((GetFrameDepth() == 1U) || (GetFrameDepth() == 0U));
+      // the native method.
+      // The top method is a runtime method, the native method comes next.
+      CHECK_EQ(GetFrameDepth(), 1U);
       callee_method_ = method;
       return true;
     } else if (!single_frame_deopt_ &&