General code cleanup of ChromeClient-related classes.

Source/core/svg/graphics/SVGImageChromeClient.h

 /*
  * Copyright (C) 2012 Apple 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 22 matching lines @@
 #include "platform/Timer.h"
 
 namespace blink {
 
 class SVGImage;
 
 class SVGImageChromeClient final : public EmptyChromeClient {
     WTF_MAKE_NONCOPYABLE(SVGImageChromeClient); WTF_MAKE_FAST_ALLOCATED(SVGImageChromeClient);
 public:
     explicit SVGImageChromeClient(SVGImage*);
-    virtual bool isSVGImageChromeClient() const override;
+    bool isSVGImageChromeClient() const override;

[yosin_UTC9, 2015/06/03 07:15:27]

Q: Do you recommend |override| than |final| in |final| class?

[tkent, 2015/06/03 07:22:19]

I have no opinion about it.
Google C++ style guide doesn't define it.  So either of |override| or |final|
should be ok.

 
     SVGImage* image() const { return m_image; }
 
 private:
-    virtual void chromeDestroyed() override;
-    virtual void invalidateRect(const IntRect&) override;
-    virtual void scheduleAnimation() override;
+    void chromeDestroyed() override;
+    void invalidateRect(const IntRect&) override;
+    void scheduleAnimation() override;
 
     void animationTimerFired(Timer<SVGImageChromeClient>*);
 
     SVGImage* m_image;
     Timer<SVGImageChromeClient> m_animationTimer;
 };
 
 DEFINE_TYPE_CASTS(SVGImageChromeClient, ChromeClient, client, client->isSVGImageChromeClient(), client.isSVGImageChromeClient());
 
 }
 
 #endif // SVGImageChromeClient_h