RawResourceTest: Use loadingTaskRunner to be balanced with Resource

third_party/WebKit/Source/core/fetch/RawResourceTest.cpp

 /*
  * Copyright (c) 2013, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 128 matching lines @@
   Vector<char> m_data;
 };
 
 // This client adds another client when notified.
 class AddingClient final : public GarbageCollectedFinalized<AddingClient>,
                            public RawResourceClient {
   USING_GARBAGE_COLLECTED_MIXIN(AddingClient);
 
  public:
   AddingClient(DummyClient* client, Resource* resource)
-      : m_dummyClient(client),
-        m_resource(resource),
-        m_removeClientTimer(this, &AddingClient::removeClient) {}
+      : m_dummyClient(client), m_resource(resource) {}
 
   ~AddingClient() override {}
 
   // ResourceClient implementation.
   void notifyFinished(Resource* resource) override {
     // First schedule an asynchronous task to remove the client.
-    // We do not expect the client to be called.
-    m_removeClientTimer.startOneShot(0, BLINK_FROM_HERE);
+    // We do not expect a client to be called if the client is removed before
+    // a callback invocation task queued inside addClient() is scheduled.
+    Platform::current()
+        ->currentThread()
+        ->scheduler()
+        ->loadingTaskRunner()
+        ->postTask(BLINK_FROM_HERE, WTF::bind(&AddingClient::removeClient,
+                                              wrapPersistent(this)));
     resource->addClient(m_dummyClient);
   }
   String debugName() const override { return "AddingClient"; }
 
-  void removeClient(TimerBase* timer) {
-    m_resource->removeClient(m_dummyClient);
-  }
+  void removeClient() { m_resource->removeClient(m_dummyClient); }
 
   DEFINE_INLINE_VIRTUAL_TRACE() {
     visitor->trace(m_dummyClient);
     visitor->trace(m_resource);
     RawResourceClient::trace(visitor);
   }
 
  private:
   Member<DummyClient> m_dummyClient;
   Member<Resource> m_resource;
-  Timer<AddingClient> m_removeClientTimer;
 };
 
 TEST(RawResourceTest, RevalidationSucceeded) {
   Resource* resource =
       RawResource::create(ResourceRequest("data:text/html,"), Resource::Raw);
   ResourceResponse response;
   response.setHTTPStatusCode(200);
   resource->responseReceived(response, nullptr);
   const char data[5] = "abcd";
   resource->appendData(data, 4);
@@ skipping 369 matching lines @@
 
 TEST(RawResourceTest, CanReuseDevToolsEmulateNetworkConditionsClientIdHeader) {
   ResourceRequest request("data:text/html,");
   request.setHTTPHeaderField(
       HTTPNames::X_DevTools_Emulate_Network_Conditions_Client_Id, "Foo");
   Resource* raw = RawResource::create(request, Resource::Raw);
   EXPECT_TRUE(raw->canReuse(ResourceRequest("data:text/html,")));
 }
 
 }  // namespace blink