Make createServiceWorkerProvider stub smarter

public/web/WebFrameClient.h

Index: public/web/WebFrameClient.h
diff --git a/public/web/WebFrameClient.h b/public/web/WebFrameClient.h
index e2e2ce57967c13cb686ea614d2c5c5639d8dfaf0..fe808efbc1ff55d0886f39f2cf79e631d0c0635c 100644
--- a/public/web/WebFrameClient.h
+++ b/public/web/WebFrameClient.h
@@ -94,7 +94,12 @@ public:
     virtual WebApplicationCacheHost* createApplicationCacheHost(WebFrame*, WebApplicationCacheHostClient*) { return 0; }
 
     // May return null. Takes ownership of the client.
-    virtual WebServiceWorkerProvider* createServiceWorkerProvider(WebFrame*, WebServiceWorkerProviderClient*) { return 0; }
+    virtual WebServiceWorkerProvider* createServiceWorkerProvider(WebFrame*, WebServiceWorkerProviderClient* client )
+    {
+        if (client)
+            delete client;
+        return 0;

[michaeln, 2013/10/24 22:53:37]

might read better as { delete client; return 0; } all on the same line, there's
no harm in calling delete on a null ptr

[alecflett, 2013/10/24 23:06:14]

oh wow, sure enough. apparently the last time I manually 'delete'ed a pointer in
C++ was 2003, when this wasn't true :)

+    }
 
 
     // Services ------------------------------------------------------------