Fix virtual/override/final usage in Source/core/svg/.

Fix virtual/override/final usage in Source/core/svg/.

- We should specify just one of them to a function.
  This was done by the following command.
  % sed -i "" -E "s/virtual (.*) final/\1 final/g;
    s/virtual (.*) override/\1 override/g;
    s/override final/final/g" <files>
- Non-base virtual destructors should have |override| keywords.

This CL has no behavior changes.

BUG=417463
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198464

[tkent, 2015-07-08 04:51:20]

tkent@chromium.org changed reviewers:
+ yosin@chromium.org

[tkent, 2015-07-08 04:51:22]

yosin@, please have a look at this.

[yosin_UTC9, 2015-07-08 05:01:33]

rs lgtm

https://codereview.chromium.org/1212253012/diff/1/Source/core/svg/SVGPathSegL...
File Source/core/svg/SVGPathSegLinetoHorizontalAbs.h (right):

https://codereview.chromium.org/1212253012/diff/1/Source/core/svg/SVGPathSegL...
Source/core/svg/SVGPathSegLinetoHorizontalAbs.h:47: String pathSegTypeAsLetter()
const override { return "H"; }
FYI: It seems |pathSegTypeAsLetter()| is used only in V8 bindiing as

v8SetReturnValueString(info, impl->pathSegTypeAsLetter(), info.GetIsolate());

We may not need to construct |String| object.

[yosin_UTC9, 2015-07-08 05:01:41]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-07-08 05:02:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1212253012/1

[commit-bot: I haz the power, 2015-07-08 05:06:57]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198464