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

Source/web/AssociatedURLLoader.cpp

 /*
  * Copyright (C) 2010, 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 106 matching lines @@
             m_blockedHeaders.remove(*it);
     }
 
     return m_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&);
 
     // Enables forwarding of error notifications to the WebURLLoaderClient. These must be
     // deferred until after the call to AssociatedURLLoader::loadAsynchronously() completes.
     void enableErrorNotifications();
 
     // Stops loading and releases the DocumentThreadableLoader as early as possible.
     void clearClient() { m_client = 0; }
@@ skipping 213 matching lines @@
         m_loader->cancel();
 }
 
 void AssociatedURLLoader::setDefersLoading(bool defersLoading)
 {
     if (m_loader)
         m_loader->setDefersLoading(defersLoading);
 }
 
 } // namespace blink