Rename StackSamplingBrowserTest => ThreadProfilerBrowserTest

Updates the name to be consistent with the source and tests in
//chrome/common/profiler.

Change-Id: Ic181aa3a67435a2af3e3e4592d06f5492ebb1f37
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2437603
Commit-Queue: Mike Wittman <wittman@chromium.org>
Reviewed-by: Etienne Pierre-Doray <etiennep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811842}

chrome/common/profiler/stack_sampling_browsertest.cc → chrome/common/profiler/thread_profiler_browsertest.cc

@@ -106,7 +106,7 @@ bool MatchesProfile(metrics::SampledProfile::TriggerEvent trigger_event,
          profile.process() == process && profile.thread() == thread;
 }
 
-class StackSamplingBrowserTest : public PlatformBrowserTest {
+class ThreadProfilerBrowserTest : public PlatformBrowserTest {
  public:
   void SetUp() override {
     // Arrange to intercept the CPU profiles at the time they're provided to the
@@ -173,56 +173,56 @@ bool WaitForProfile(metrics::SampledProfile::TriggerEvent trigger_event,
 // processes/threads. We've seen multiple breakages previously where profiles
 // were dropped as a result of bugs introduced by mojo refactorings.
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, BrowserProcessMainThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, BrowserProcessMainThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::BROWSER_PROCESS, metrics::MAIN_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, BrowserProcessIOThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, BrowserProcessIOThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::BROWSER_PROCESS, metrics::IO_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, GpuProcessMainThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, GpuProcessMainThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::GPU_PROCESS, metrics::MAIN_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, GpuProcessIOThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, GpuProcessIOThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::GPU_PROCESS, metrics::IO_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, GpuProcessCompositorThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, GpuProcessCompositorThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::GPU_PROCESS, metrics::COMPOSITOR_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, RendererProcessMainThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, RendererProcessMainThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::RENDERER_PROCESS, metrics::MAIN_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest, RendererProcessIOThread) {
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest, RendererProcessIOThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
   EXPECT_TRUE(WaitForProfile(metrics::SampledProfile::PROCESS_STARTUP,
                              metrics::RENDERER_PROCESS, metrics::IO_THREAD));
 }
 
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest,
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest,
                        RendererProcessCompositorThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";
@@ -238,7 +238,7 @@ IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest,
 #else
 #define MAYBE_NetworkServiceProcessIOThread NetworkServiceProcessIOThread
 #endif
-IN_PROC_BROWSER_TEST_F(StackSamplingBrowserTest,
+IN_PROC_BROWSER_TEST_F(ThreadProfilerBrowserTest,
                        MAYBE_NetworkServiceProcessIOThread) {
   if (ShouldSkipTestForMacOS11())
     GTEST_SKIP() << "Stack sampler is not supported on macOS 11.";

chrome/test/BUILD.gn

@@ -556,7 +556,7 @@ if (is_android) {
     # "strip_debug" config that strips the debug info, on android_browsertests
     # apk.
     if (enable_arm_cfi_table) {
-      sources += [ "../common/profiler/stack_sampling_browsertest.cc" ]
+      sources += [ "../common/profiler/thread_profiler_browsertest.cc" ]
       deps += [
         "//chrome/android/modules/stack_unwinder/internal:java",
         "//chrome/android/modules/stack_unwinder/internal:stack_unwinder_for_testing",
@@ -2840,7 +2840,7 @@ if (!is_android) {
         "../browser/ui/cocoa/task_manager_mac_browsertest.mm",
         "../browser/ui/cocoa/touchbar/browser_window_touch_bar_controller_browsertest.mm",
         "../browser/ui/views/frame/browser_non_client_frame_view_mac_browsertest.cc",
-        "../common/profiler/stack_sampling_browsertest.cc",
+        "../common/profiler/thread_profiler_browsertest.cc",
 
         # TODO(crbug/845389): Re-Enable the following, which were temporarily
         # omitted from the build, but are still in use.
@@ -2877,7 +2877,7 @@ if (!is_android) {
       ]
 
       if (target_cpu == "x64") {
-        sources += [ "../common/profiler/stack_sampling_browsertest.cc" ]
+        sources += [ "../common/profiler/thread_profiler_browsertest.cc" ]
       }
 
       configs += [ "//build/config/win:delayloads" ]