Remove support for TRACE_EVENT_IF_LONGER_THANx macros

base/debug/trace_event_unittest.cc

 // Copyright (c) 2012 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 "base/debug/trace_event_unittest.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/json/json_reader.h"
@@ skipping 320 matching lines @@
     TRACE_EVENT_BEGIN2("all", "TRACE_EVENT_BEGIN2 call",
                        "name1", "value1",
                        "name2", "value2");
 
     TRACE_EVENT_END0("all", "TRACE_EVENT_END0 call");
     TRACE_EVENT_END1("all", "TRACE_EVENT_END1 call", "name1", "value1");
     TRACE_EVENT_END2("all", "TRACE_EVENT_END2 call",
                      "name1", "value1",
                      "name2", "value2");
 
-    TRACE_EVENT_IF_LONGER_THAN0(0, "all", "TRACE_EVENT_IF_LONGER_THAN0 call");
-    TRACE_EVENT_IF_LONGER_THAN1(0, "all", "TRACE_EVENT_IF_LONGER_THAN1 call",
-                                "name1", "value1");
-    TRACE_EVENT_IF_LONGER_THAN2(0, "all", "TRACE_EVENT_IF_LONGER_THAN2 call",
-                                "name1", "value1",
-                                "name2", "value2");
-
     TRACE_EVENT_ASYNC_BEGIN0("all", "TRACE_EVENT_ASYNC_BEGIN0 call", 5);
     TRACE_EVENT_ASYNC_BEGIN1("all", "TRACE_EVENT_ASYNC_BEGIN1 call", 5,
                              "name1", "value1");
     TRACE_EVENT_ASYNC_BEGIN2("all", "TRACE_EVENT_ASYNC_BEGIN2 call", 5,
                              "name1", "value1",
                              "name2", "value2");
 
     TRACE_EVENT_ASYNC_STEP0("all", "TRACE_EVENT_ASYNC_STEP0 call",
                                   5, "step1");
     TRACE_EVENT_ASYNC_STEP1("all", "TRACE_EVENT_ASYNC_STEP1 call",
@@ skipping 92 matching lines @@
   EXPECT_FIND_("TRACE_EVENT_END0 call");
   EXPECT_FIND_("TRACE_EVENT_END1 call");
   EXPECT_SUB_FIND_("name1");
   EXPECT_SUB_FIND_("value1");
   EXPECT_FIND_("TRACE_EVENT_END2 call");
   EXPECT_SUB_FIND_("name1");
   EXPECT_SUB_FIND_("value1");
   EXPECT_SUB_FIND_("name2");
   EXPECT_SUB_FIND_("value2");
 
-  EXPECT_FIND_("TRACE_EVENT_IF_LONGER_THAN0 call");
-  EXPECT_FIND_("TRACE_EVENT_IF_LONGER_THAN1 call");
-  EXPECT_SUB_FIND_("name1");
-  EXPECT_SUB_FIND_("value1");
-  EXPECT_FIND_("TRACE_EVENT_IF_LONGER_THAN2 call");
-  EXPECT_SUB_FIND_("name1");
-  EXPECT_SUB_FIND_("value1");
-  EXPECT_SUB_FIND_("name2");
-  EXPECT_SUB_FIND_("value2");
-
   EXPECT_FIND_("TRACE_EVENT_ASYNC_BEGIN0 call");
   EXPECT_SUB_FIND_("id");
   EXPECT_SUB_FIND_("5");
   EXPECT_FIND_("TRACE_EVENT_ASYNC_BEGIN1 call");
   EXPECT_SUB_FIND_("id");
   EXPECT_SUB_FIND_("5");
   EXPECT_SUB_FIND_("name1");
   EXPECT_SUB_FIND_("value1");
   EXPECT_FIND_("TRACE_EVENT_ASYNC_BEGIN2 call");
   EXPECT_SUB_FIND_("id");
@@ skipping 370 matching lines @@
   TRACE_EVENT_INSTANT0("inc_wildchar_bla_end", "included");
   TRACE_EVENT_INSTANT0("cat1", "included");
   TRACE_EVENT_INSTANT0("cat2", "included");
   EndTraceAndFlush();
   EXPECT_TRUE(FindMatchingValue("cat", "inc_wildchar_bla_end"));
   EXPECT_TRUE(FindMatchingValue("cat", "cat1"));
   EXPECT_TRUE(FindMatchingValue("cat", "cat2"));
   EXPECT_FALSE(FindMatchingValue("name", "not_inc"));
 }
 
-// Simple Test for time threshold events.
-TEST_F(TraceEventTestFixture, DataCapturedThreshold) {
-  ManualTestSetUp();
-  BeginTrace();
-
-  // Test that events at the same level are properly filtered by threshold.
-  {
-    TRACE_EVENT_IF_LONGER_THAN0(100, "time", "threshold 100");
-    TRACE_EVENT_IF_LONGER_THAN0(1000, "time", "threshold 1000");
-    TRACE_EVENT_IF_LONGER_THAN0(10000, "time", "threshold 10000");
-    // 100+ seconds to avoid flakiness.
-    TRACE_EVENT_IF_LONGER_THAN0(100000000, "time", "threshold long1");
-    TRACE_EVENT_IF_LONGER_THAN0(200000000, "time", "threshold long2");
-    HighResSleepForTraceTest(base::TimeDelta::FromMilliseconds(20));
-  }
-
-  // Test that a normal nested event remains after it's parent event is dropped.
-  {
-    TRACE_EVENT_IF_LONGER_THAN0(1000000, "time", "2threshold10000");
-    {
-      TRACE_EVENT0("time", "nonthreshold1");
-    }
-  }
-
-  // Test that parent thresholded events are dropped while some nested events
-  // remain.
-  {
-    TRACE_EVENT0("time", "nonthreshold3");
-    {
-      TRACE_EVENT_IF_LONGER_THAN0(200000000, "time", "3thresholdlong2");
-      {
-        TRACE_EVENT_IF_LONGER_THAN0(100000000, "time", "3thresholdlong1");
-        {
-          TRACE_EVENT_IF_LONGER_THAN0(10000, "time", "3threshold10000");
-          {
-            TRACE_EVENT_IF_LONGER_THAN0(1000, "time", "3threshold1000");
-            {
-              TRACE_EVENT_IF_LONGER_THAN0(100, "time", "3threshold100");
-              HighResSleepForTraceTest(base::TimeDelta::FromMilliseconds(20));
-            }
-          }
-        }
-      }
-    }
-  }
-
-  // Test that child thresholded events are dropped while some parent events
-  // remain.
-  {
-    TRACE_EVENT0("time", "nonthreshold4");
-    {
-      TRACE_EVENT_IF_LONGER_THAN0(100, "time", "4threshold100");
-      {
-        TRACE_EVENT_IF_LONGER_THAN0(1000, "time", "4threshold1000");
-        {
-          TRACE_EVENT_IF_LONGER_THAN0(10000, "time", "4threshold10000");
-          {
-            TRACE_EVENT_IF_LONGER_THAN0(100000000, "time",
-                                        "4thresholdlong1");
-            {
-              TRACE_EVENT_IF_LONGER_THAN0(200000000, "time",
-                                          "4thresholdlong2");
-              HighResSleepForTraceTest(base::TimeDelta::FromMilliseconds(20));
-            }
-          }
-        }
-      }
-    }
-  }
-
-  EndTraceAndFlush();
-
-#define EXPECT_FIND_BE_(str) \
-  EXPECT_TRUE(FindNamePhase(str, "B")); \
-  EXPECT_TRUE(FindNamePhase(str, "E"))
-#define EXPECT_NOT_FIND_BE_(str) \
-  EXPECT_FALSE(FindNamePhase(str, "B")); \
-  EXPECT_FALSE(FindNamePhase(str, "E"))
-
-  EXPECT_FIND_BE_("threshold 100");
-  EXPECT_FIND_BE_("threshold 1000");
-  EXPECT_FIND_BE_("threshold 10000");
-  EXPECT_NOT_FIND_BE_("threshold long1");
-  EXPECT_NOT_FIND_BE_("threshold long2");
-
-  EXPECT_NOT_FIND_BE_("2threshold10000");
-  EXPECT_FIND_BE_("nonthreshold1");
-
-  EXPECT_FIND_BE_("nonthreshold3");
-  EXPECT_FIND_BE_("3threshold100");
-  EXPECT_FIND_BE_("3threshold1000");
-  EXPECT_FIND_BE_("3threshold10000");
-  EXPECT_NOT_FIND_BE_("3thresholdlong1");
-  EXPECT_NOT_FIND_BE_("3thresholdlong2");
-
-  EXPECT_FIND_BE_("nonthreshold4");
-  EXPECT_FIND_BE_("4threshold100");
-  EXPECT_FIND_BE_("4threshold1000");
-  EXPECT_FIND_BE_("4threshold10000");
-  EXPECT_NOT_FIND_BE_("4thresholdlong1");
-  EXPECT_NOT_FIND_BE_("4thresholdlong2");
-}
 
 // Test EVENT_WATCH_NOTIFICATION
 TEST_F(TraceEventTestFixture, EventWatchNotification) {
   ManualTestSetUp();
 
   // Basic one occurrence.
   BeginTrace();
   TraceLog::GetInstance()->SetWatchEvent("cat", "event");
   TRACE_EVENT_INSTANT0("cat", "event");
   EndTraceAndFlush();
@@ skipping 479 matching lines @@
   Clear();
 
   trace_buffer_.Start();
   trace_buffer_.AddFragment("bla1,bla2,bla3,bla4");
   trace_buffer_.Finish();
   EXPECT_STREQ(json_output_.json_output.c_str(), "[bla1,bla2,bla3,bla4]");
 }
 
 }  // namespace debug
 }  // namespace base