Update fetch / loader classes to use OVERRIDE / FINAL when needed

Source/web/AssociatedURLLoader.cpp

Index: Source/web/AssociatedURLLoader.cpp
diff --git a/Source/web/AssociatedURLLoader.cpp b/Source/web/AssociatedURLLoader.cpp
index 9eaf5e8a4cf203dd237eee852b6d80a0f4215bf8..154e02578357880476fd4ab588f05a63451b0600 100644
--- a/Source/web/AssociatedURLLoader.cpp
+++ b/Source/web/AssociatedURLLoader.cpp
@@ -124,23 +124,21 @@ const HTTPHeaderSet& HTTPResponseHeaderValidator::blockedHeaders()
 
 // This class bridges the interface differences between WebCore and WebKit loader clients.
 // It forwards its ThreadableLoaderClient notifications to a WebURLLoaderClient.
-class AssociatedURLLoader::ClientAdapter : public DocumentThreadableLoaderClient {
+class AssociatedURLLoader::ClientAdapter FINAL : public DocumentThreadableLoaderClient {
     WTF_MAKE_NONCOPYABLE(ClientAdapter);
 public:
     static PassOwnPtr<ClientAdapter> create(AssociatedURLLoader*, WebURLLoaderClient*, const WebURLLoaderOptions&);
 
-    virtual void didSendData(unsigned long long /*bytesSent*/, unsigned long long /*totalBytesToBeSent*/);
-    virtual void willSendRequest(ResourceRequest& /*newRequest*/, const ResourceResponse& /*redirectResponse*/);
+    virtual void didSendData(unsigned long long /*bytesSent*/, unsigned long long /*totalBytesToBeSent*/) OVERRIDE;
+    virtual void willSendRequest(ResourceRequest& /*newRequest*/, const ResourceResponse& /*redirectResponse*/) OVERRIDE;
 
-    virtual void didReceiveResponse(unsigned long, const ResourceResponse&);
-    virtual void didDownloadData(int /*dataLength*/);
-    virtual void didReceiveData(const char*, int /*dataLength*/);
-    virtual void didReceiveCachedMetadata(const char*, int /*dataLength*/);
-    virtual void didFinishLoading(unsigned long /*identifier*/, double /*finishTime*/);
-    virtual void didFail(const ResourceError&);
-    virtual void didFailRedirectCheck();
-
-    virtual bool isDocumentThreadableLoaderClient() { return true; }

[Inactive, 2014/01/10 19:28:09]

If you prefer, I can keep the OVERRIDE / FINAL for this web class out of this
patch. The only thing I need is to remove this
isDocumentThreadableLoaderClient() (it already returns true in the parent and
I'd to mark the parent implementation as FINAL).

FYI, OVERRIDE is already used quite a lot in web:
$ git grep OVERRIDE Source/web/ | wc -l
491

+    virtual void didReceiveResponse(unsigned long, const ResourceResponse&) OVERRIDE;
+    virtual void didDownloadData(int /*dataLength*/) OVERRIDE;
+    virtual void didReceiveData(const char*, int /*dataLength*/) OVERRIDE;
+    virtual void didReceiveCachedMetadata(const char*, int /*dataLength*/) OVERRIDE;
+    virtual void didFinishLoading(unsigned long /*identifier*/, double /*finishTime*/) OVERRIDE;
+    virtual void didFail(const ResourceError&) OVERRIDE;
+    virtual void didFailRedirectCheck() OVERRIDE;
 
     // Sets an error to be reported back to the client, asychronously.
     void setDelayedError(const ResourceError&);