Make the WebURLLoaderClients in the pepper code destroy (or have some guard) the loader in didFail()

content/renderer/pepper/pepper_plugin_instance_impl.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_RENDERER_PEPPER_PEPPER_PLUGIN_INSTANCE_IMPL_H_
 #define CONTENT_RENDERER_PEPPER_PEPPER_PLUGIN_INSTANCE_IMPL_H_
 
 #include <list>
 #include <set>
 #include <string>
@@ skipping 534 matching lines @@
   // Class to record document load notifications and play them back once the
   // real document loader becomes available. Used only by external instances.
   class ExternalDocumentLoader : public blink::WebURLLoaderClient {
    public:
     ExternalDocumentLoader();
     virtual ~ExternalDocumentLoader();
 
     void ReplayReceivedData(WebURLLoaderClient* document_loader);
 
     // blink::WebURLLoaderClient implementation.
-    virtual void didReceiveData(blink::WebURLLoader* loader,
-                                const char* data,
-                                int data_length,
-                                int encoded_data_length);
-    virtual void didFinishLoading(blink::WebURLLoader* loader,
-                                  double finish_time,
-                                  int64_t total_encoded_data_length);
-    virtual void didFail(blink::WebURLLoader* loader,
-                         const blink::WebURLError& error);
+    void didReceiveData(blink::WebURLLoader* loader,
+                        const char* data,
+                        int data_length,
+                        int encoded_data_length) override;
+    void didFinishLoading(blink::WebURLLoader* loader,
+                          double finish_time,
+                          int64_t total_encoded_data_length) override;
+    void didFail(blink::WebURLLoader* loader,
+                 const blink::WebURLError& error) override;
 
    private:
     std::list<std::string> data_;
     bool finished_loading_;
     scoped_ptr<blink::WebURLError> error_;
   };
 
   // Implements PPB_Gamepad_API. This is just to avoid having an excessive
   // number of interfaces implemented by PepperPluginInstanceImpl.
   class GamepadImpl : public ppapi::thunk::PPB_Gamepad_API,
@@ skipping 345 matching lines @@
   // view change events.
   base::WeakPtrFactory<PepperPluginInstanceImpl> view_change_weak_ptr_factory_;
   base::WeakPtrFactory<PepperPluginInstanceImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperPluginInstanceImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_PEPPER_PEPPER_PLUGIN_INSTANCE_IMPL_H_