Remove FetchContext::platform_probe_sink_

It is no longer used.

Bug: 914739
Change-Id: Ic5d4df6954ea1bee276b0ac41735c4c1bae2fba4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1619523Reviewed-by: Alexei Filippov <alph@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662527}

third_party/blink/renderer/core/probe/core_probes.h

@@ -34,6 +34,7 @@
 #include "third_party/blink/renderer/core/core_export.h"
 #include "third_party/blink/renderer/core/dom/document.h"
 #include "third_party/blink/renderer/core/frame/local_frame.h"
+#include "third_party/blink/renderer/platform/loader/fetch/resource.h"
 #include "third_party/blink/renderer/platform/probe/platform_probes.h"
 
 namespace network {

third_party/blink/renderer/platform/loader/fetch/fetch_context.cc

@@ -31,8 +31,6 @@
 #include "third_party/blink/renderer/platform/loader/fetch/fetch_context.h"
 
 #include "third_party/blink/renderer/platform/loader/fetch/resource_fetcher.h"
-#include "third_party/blink/renderer/platform/platform_probe_sink.h"
-#include "third_party/blink/renderer/platform/probe/platform_trace_events_agent.h"
 
 namespace blink {
 
@@ -52,16 +50,6 @@ FetchContext& FetchContext::NullInstance() {
   return *MakeGarbageCollected<NullFetchContext>();
 }
 
-FetchContext::FetchContext()
-    : platform_probe_sink_(MakeGarbageCollected<PlatformProbeSink>()) {
-  platform_probe_sink_->AddPlatformTraceEvents(
-      MakeGarbageCollected<PlatformTraceEventsAgent>());
-}
-
-void FetchContext::Trace(blink::Visitor* visitor) {
-  visitor->Trace(platform_probe_sink_);
-}
-
 void FetchContext::AddAdditionalRequestHeaders(ResourceRequest&) {}
 
 mojom::FetchCacheMode FetchContext::ResourceRequestCachePolicy(

third_party/blink/renderer/platform/loader/fetch/fetch_context.h

@@ -59,7 +59,6 @@ namespace blink {
 enum class ResourceType : uint8_t;
 class ClientHintsPreferences;
 class KURL;
-class PlatformProbeSink;
 class ResourceTimingInfo;
 class WebScopedVirtualTimePauser;
 
@@ -73,13 +72,13 @@ class WebScopedVirtualTimePauser;
 class PLATFORM_EXPORT FetchContext
     : public GarbageCollectedFinalized<FetchContext> {
  public:
-  FetchContext();
+  FetchContext() = default;
 
   static FetchContext& NullInstance();
 
   virtual ~FetchContext() = default;
 
-  virtual void Trace(blink::Visitor*);
+  virtual void Trace(blink::Visitor*) {}
 
   virtual void AddAdditionalRequestHeaders(ResourceRequest&);
 
@@ -140,10 +139,6 @@ class PLATFORM_EXPORT FetchContext
                                        const FetchParameters::ResourceWidth&,
                                        ResourceRequest&);
 
-  PlatformProbeSink* GetPlatformProbeSink() const {
-    return platform_probe_sink_;
-  }
-
   // Called when the underlying context is detached. Note that some
   // FetchContexts continue working after detached (e.g., for fetch() operations
   // with "keepalive" specified).
@@ -161,8 +156,6 @@ class PLATFORM_EXPORT FetchContext
   }
 
  private:
-  Member<PlatformProbeSink> platform_probe_sink_;
-
   DISALLOW_COPY_AND_ASSIGN(FetchContext);
 };
 

third_party/blink/renderer/platform/loader/fetch/resource_fetcher.cc

@@ -1860,10 +1860,6 @@ bool ResourceFetcher::StartLoad(Resource* resource) {
 
     ResourceResponse response;
 
-    blink::probe::PlatformSendRequest probe(&Context(), resource->InspectorId(),
-                                            request, response,
-                                            resource->Options().initiator_info);
-
     resource->VirtualTimePauser().PauseVirtualTime();
     if (resource_load_observer_) {
       DCHECK(!IsDetached());

third_party/blink/renderer/platform/probe/platform_probes.h

@@ -5,15 +5,12 @@
 #ifndef THIRD_PARTY_BLINK_RENDERER_PLATFORM_PROBE_PLATFORM_PROBES_H_
 #define THIRD_PARTY_BLINK_RENDERER_PLATFORM_PROBE_PLATFORM_PROBES_H_
 
-#include "third_party/blink/renderer/platform/loader/fetch/fetch_context.h"
 #include "third_party/blink/renderer/platform/platform_export.h"
+#include "third_party/blink/renderer/platform/wtf/allocator.h"
 #include "third_party/blink/renderer/platform/wtf/time.h"
 
 namespace blink {
 
-class FetchContext;
-class PlatformProbeSink;
-
 namespace probe {
 
 class PLATFORM_EXPORT ProbeBase {
@@ -29,10 +26,6 @@ class PLATFORM_EXPORT ProbeBase {
   mutable TimeTicks end_time_;
 };
 
-inline PlatformProbeSink* ToPlatformProbeSink(FetchContext* context) {
-  return context->GetPlatformProbeSink();
-}
-
 }  // namespace probe
 }  // namespace blink
 

third_party/blink/renderer/platform/probe/platform_probes.json5

@@ -9,11 +9,5 @@
     ]
   },
   observers: {
-    PlatformTraceEvents: {
-      class: "PlatformTraceEventsAgent",
-      probes: [
-        "PlatformSendRequest",
-      ]
-    }
   }
 }

third_party/blink/renderer/platform/probe/platform_probes.pidl

@@ -3,10 +3,4 @@
 // found in the LICENSE file.
 
 interface PlatformProbes {
-  class FetchContext;
-  class ResourceRequest;
-  class ResourceResponse;
-  struct FetchInitiatorInfo;
-
-  PlatformSendRequest([Keep] FetchContext*, uint64_t identifier, ResourceRequest&, const ResourceResponse& redirect_response, const FetchInitiatorInfo&);
 }