Fix double delete in url_loader_impl.cc

mojo/services/network/url_loader_impl.cc

 // Copyright 2014 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 "mojo/services/network/url_loader_impl.h"
 
 #include <memory>
 
 #include "base/memory/scoped_vector.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 250 matching lines @@
 URLLoaderImpl::URLLoaderImpl(NetworkContext* context,
                              InterfaceRequest<URLLoader> request,
                              std::vector<URLLoaderInterceptorPtr> interceptors)
     : context_(context),
       interceptors_(std::move(interceptors)),
       response_body_buffer_size_(0),
       auto_follow_redirects_(true),
       current_fetcher_id_(0),
       binding_(this, request.Pass()) {
   for (auto& interceptor : interceptors_) {
-    interceptor.set_connection_error_handler([this]() { OnConnectionError(); });
+    interceptor.set_connection_error_handler([this]() { delete this; });
   }
   interceptor_index_ = interceptors_.size() - 1;
   binding_.set_connection_error_handler([this]() { OnConnectionError(); });
   context_->RegisterURLLoader(this);
 }
 
 URLLoaderImpl::~URLLoaderImpl() {
   context_->DeregisterURLLoader(this);
 }
 
@@ skipping 294 matching lines @@
   if ((*it)->id() == current_fetcher_id_) {
     last_status_ = fetcher->QueryStatus();
     last_status_->is_loading = false;
   }
   body_fetchers_.erase(it);
   if (body_fetchers_.empty() and !binding_.is_bound())
     delete this;
 }
 
 }  // namespace mojo