[Fetch, Loader] Expect on-heap objects will never get destroyed with a reference

third_party/WebKit/Source/core/loader/ThreadableLoader.h

Index: third_party/WebKit/Source/core/loader/ThreadableLoader.h
diff --git a/third_party/WebKit/Source/core/loader/ThreadableLoader.h b/third_party/WebKit/Source/core/loader/ThreadableLoader.h
index ef75a632ee68fcd34283f3f48b3770120b0dfd47..d2e1a91abd731c0d9f29a9345c057a045abbb1b0 100644
--- a/third_party/WebKit/Source/core/loader/ThreadableLoader.h
+++ b/third_party/WebKit/Source/core/loader/ThreadableLoader.h
@@ -131,8 +131,6 @@ class CORE_EXPORT ThreadableLoader
   WTF_MAKE_NONCOPYABLE(ThreadableLoader);
 
  public:
-  // ThreadableLoaderClient methods may not destroy the ThreadableLoader
-  // instance in them.
   static void loadResourceSynchronously(ExecutionContext&,
                                         const ResourceRequest&,
                                         ThreadableLoaderClient&,
@@ -168,11 +166,6 @@ class CORE_EXPORT ThreadableLoader
   // or didFail.*() methods have not been called yet. (didFail() may be
   // called with a ResourceError with isCancellation() returning true
   // also for cancellation happened inside the loader.)
-  //
-  // ThreadableLoaderClient methods:

[hiroshige, 2016/10/21 05:45:14]

Isn't the part of "ThreadableLoaderClient methods may call cancel()" still
valid?

[yhirano, 2016/10/26 11:09:34]

You're right, thanks.

-  // - may call cancel()
-  // - can destroy the ThreadableLoader instance in them (by clearing
-  //   std::unique_ptr<ThreadableLoader>).
   static ThreadableLoader* create(ExecutionContext&,
                                   ThreadableLoaderClient*,
                                   const ThreadableLoaderOptions&,