[Tracing] Remove deprecated AddTraceEvent

This should land after the chromium CL that starts using the new AddTraceEvent lands
(https://codereview.chromium.org/1742603004/)

BUG=4565
LOG=N

Review URL: https://codereview.chromium.org/1744733002

Cr-Commit-Position: refs/heads/master@{#34412}

include/v8-platform.h

@@ -155,21 +155,6 @@ class Platform {
       const char* scope, uint64_t id, uint64_t bind_id, int32_t num_args,
       const char** arg_names, const uint8_t* arg_types,
       const uint64_t* arg_values, unsigned int flags) {
-    return AddTraceEvent(phase, category_enabled_flag, name, id, bind_id,
-                         num_args, arg_names, arg_types, arg_values, flags);
-  }
-
-  /**
-   * This method will soon be depreacted in favor of the version with the 'const
-   * char* scope' parameter. If you do not already override this method, please
-   * implement the other version instead.
-   */
-  // TODO(fmeawad): Remove once all embedders implement the scope version.
-  virtual uint64_t AddTraceEvent(
-      char phase, const uint8_t* category_enabled_flag, const char* name,
-      uint64_t id, uint64_t bind_id, int32_t num_args, const char** arg_names,
-      const uint8_t* arg_types, const uint64_t* arg_values,
-      unsigned int flags) {
     return 0;
   }
 

src/d8.cc

@@ -140,15 +140,6 @@ class PredictablePlatform : public Platform {
     return 0;
   }
 
-  // TODO(fmeawad): Remove once all embedders implement the scope version.
-  uint64_t AddTraceEvent(char phase, const uint8_t* categoryEnabledFlag,
-                         const char* name, uint64_t id, uint64_t bind_id,
-                         int numArgs, const char** argNames,
-                         const uint8_t* argTypes, const uint64_t* argValues,
-                         unsigned int flags) override {
-    return 0;
-  }
-
   void UpdateTraceEventDuration(const uint8_t* categoryEnabledFlag,
                                 const char* name, uint64_t handle) override {}
 

src/libplatform/default-platform.cc

@@ -179,15 +179,6 @@ uint64_t DefaultPlatform::AddTraceEvent(
 }
 
 
-// TODO(fmeawad): Remove once all embedders implement the scope version.
-uint64_t DefaultPlatform::AddTraceEvent(
-    char phase, const uint8_t* category_enabled_flag, const char* name,
-    uint64_t id, uint64_t bind_id, int num_args, const char** arg_names,
-    const uint8_t* arg_types, const uint64_t* arg_values, unsigned int flags) {
-  return 0;
-}
-
-
 void DefaultPlatform::UpdateTraceEventDuration(
     const uint8_t* category_enabled_flag, const char* name, uint64_t handle) {}
 

src/libplatform/default-platform.h

@@ -52,12 +52,6 @@ class DefaultPlatform : public Platform {
                          const char** arg_names, const uint8_t* arg_types,
                          const uint64_t* arg_values,
                          unsigned int flags) override;
-  // TODO(fmeawad): Remove once all embedders implement the scope version.
-  uint64_t AddTraceEvent(char phase, const uint8_t* category_enabled_flag,
-                         const char* name, uint64_t id, uint64_t bind_id,
-                         int32_t num_args, const char** arg_names,
-                         const uint8_t* arg_types, const uint64_t* arg_values,
-                         unsigned int flags) override;
   void UpdateTraceEventDuration(const uint8_t* category_enabled_flag,
                                 const char* name, uint64_t handle) override;
 

test/cctest/heap/test-incremental-marking.cc

@@ -93,15 +93,6 @@ class MockPlatform : public v8::Platform {
     return 0;
   }
 
-  // TODO(fmeawad): Remove once all embedders implement the scope version.
-  uint64_t AddTraceEvent(char phase, const uint8_t* categoryEnabledFlag,
-                         const char* name, uint64_t id, uint64_t bind_id,
-                         int numArgs, const char** argNames,
-                         const uint8_t* argTypes, const uint64_t* argValues,
-                         unsigned int flags) override {
-    return 0;
-  }
-
   void UpdateTraceEventDuration(const uint8_t* categoryEnabledFlag,
                                 const char* name, uint64_t handle) override {}
 

test/cctest/test-trace-event.cc

@@ -82,15 +82,6 @@ class MockTracingPlatform : public v8::Platform {
     return 0;
   }
 
-  // TODO(fmeawad): Remove once all embedders implement the scope version.
-  uint64_t AddTraceEvent(char phase, const uint8_t* category_enabled_flag,
-                         const char* name, uint64_t id, uint64_t bind_id,
-                         int num_args, const char** arg_names,
-                         const uint8_t* arg_types, const uint64_t* arg_values,
-                         unsigned int flags) override {
-    return 0;
-  }
-
   void UpdateTraceEventDuration(const uint8_t* category_enabled_flag,
                                 const char* name, uint64_t handle) override {}