Stack sampling profiler: remove RUNTIME_FUNCTION sanity check

base/profiler/win32_stack_frame_unwinder_unittest.cc

 // Copyright 2015 The Chromium 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 <vector>
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/profiler/win32_stack_frame_unwinder.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 357 matching lines @@
     scoped_ptr<Win32StackFrameUnwinder> unwinder = CreateUnwinder();
     CONTEXT context = {0};
     ScopedModuleHandle module;
     unwind_functions_->SetNoRuntimeFunction(image_base_for_questionable_module,
                                             &context);
     EXPECT_TRUE(unwinder->TryUnwind(&context, &module));
     EXPECT_TRUE(module.IsValid());
   }
 }
 
-// Checks that frames with RUNTIME_FUNCTION structures with nonsensical values
-// are not unwound.
-TEST_F(Win32StackFrameUnwinderTest, RuntimeFunctionSanityCheck) {
-  const DWORD64 image_base_for_sanity_check = 3072;
-  {
-    // Test the expected case: end address greater than begin address and
-    // instruction pointer between the two.
-    scoped_ptr<Win32StackFrameUnwinder> unwinder = CreateUnwinder();
-    CONTEXT context = {0};
-    ScopedModuleHandle module;
-    RUNTIME_FUNCTION runtime_function = {0};
-    runtime_function.BeginAddress = 128;
-    runtime_function.EndAddress = 512;
-    unwind_functions_->SetHasRuntimeFunction(image_base_for_sanity_check,
-                                             runtime_function, 256,
-                                             &context);
-    EXPECT_TRUE(unwinder->TryUnwind(&context, &module));
-    EXPECT_TRUE(module.IsValid());
-  }
-
-  {
-    // Test begin address greater than end address.
-    scoped_ptr<Win32StackFrameUnwinder> unwinder = CreateUnwinder();
-    CONTEXT context = {0};
-    ScopedModuleHandle module;
-    RUNTIME_FUNCTION runtime_function = {0};
-    runtime_function.BeginAddress = 512;
-    runtime_function.EndAddress = 128;
-    unwind_functions_->SetHasRuntimeFunction(image_base_for_sanity_check,
-                                             runtime_function, 256,
-                                             &context);
-    EXPECT_FALSE(unwinder->TryUnwind(&context, &module));
-  }
-
-  {
-    // Test instruction pointer before begin address.
-    scoped_ptr<Win32StackFrameUnwinder> unwinder = CreateUnwinder();
-    CONTEXT context = {0};
-    ScopedModuleHandle module;
-    RUNTIME_FUNCTION runtime_function = {0};
-    runtime_function.BeginAddress = 128;
-    runtime_function.EndAddress = 512;
-    unwind_functions_->SetHasRuntimeFunction(image_base_for_sanity_check,
-                                             runtime_function, 50,
-                                             &context);
-    EXPECT_FALSE(unwinder->TryUnwind(&context, &module));
-  }
-
-  {
-    // Test instruction pointer after end address.
-    scoped_ptr<Win32StackFrameUnwinder> unwinder = CreateUnwinder();
-    CONTEXT context = {0};
-    ScopedModuleHandle module;
-    RUNTIME_FUNCTION runtime_function = {0};
-    runtime_function.BeginAddress = 128;
-    runtime_function.EndAddress = 512;
-    unwind_functions_->SetHasRuntimeFunction(image_base_for_sanity_check,
-                                             runtime_function, 600,
-                                             &context);
-    EXPECT_FALSE(unwinder->TryUnwind(&context, &module));
-  }
-}
-
 }  // namespace base