Fix BlobBytesConsumer leak

third_party/WebKit/Source/modules/fetch/BlobBytesConsumer.cpp

 // Copyright 2016 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 "modules/fetch/BlobBytesConsumer.h"
 
 #include "core/loader/ThreadableLoader.h"
 #include "modules/fetch/BytesConsumerForDataConsumerHandle.h"
 #include "platform/blob/BlobData.h"
 #include "platform/blob/BlobRegistry.h"
@@ skipping 19 matching lines @@
       m_loader = nullptr;
     }
     m_state = PublicState::Closed;
   }
 }
 
 BlobBytesConsumer::BlobBytesConsumer(ExecutionContext* executionContext,
                                      PassRefPtr<BlobDataHandle> blobDataHandle)
     : BlobBytesConsumer(executionContext, std::move(blobDataHandle), nullptr) {}
 
-BlobBytesConsumer::~BlobBytesConsumer() {}
+BlobBytesConsumer::~BlobBytesConsumer() {
+  if (!m_blobURL.isEmpty())
+    BlobRegistry::revokePublicBlobURL(m_blobURL);
+}
 
 BytesConsumer::Result BlobBytesConsumer::beginRead(const char** buffer,
                                                    size_t* available) {
   *buffer = nullptr;
   *available = 0;
 
   if (m_state == PublicState::Closed) {
     // It's possible that |cancel| has been called before the first
     // |beginRead| call. That's why we need to check this condition
     // before checking |isClean()|.
     return Result::Done;
   }
 
   if (isClean()) {
-    KURL m_blobURL =
+    DCHECK(m_blobURL.isEmpty());
+    m_blobURL =
         BlobURL::createPublicURL(getExecutionContext()->getSecurityOrigin());
     if (m_blobURL.isEmpty()) {
       error();
     } else {
       BlobRegistry::registerPublicBlobURL(
           getExecutionContext()->getSecurityOrigin(), m_blobURL,
           m_blobDataHandle);
 
       // m_loader is non-null only in tests.
       if (!m_loader)
@@ skipping 231 matching lines @@
 void BlobBytesConsumer::clear() {
   DCHECK_NE(m_state, PublicState::ReadableOrWaiting);
   if (m_loader) {
     m_loader->cancel();
     m_loader = nullptr;
   }
   m_client = nullptr;
 }
 
 }  // namespace blink