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

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

- We should specify just one of them to a function.
- 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=198370

[tkent, 2015-07-07 06:17:39]

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

[tkent, 2015-07-07 06:17:40]

yosin@, please take a look at this.

[yosin_UTC9, 2015-07-07 06:40:09]

lgtm

Thanks for clean up!

It seems mac bot has trouble with running "isolate tests".

https://codereview.chromium.org/1218293014/diff/1/Source/core/editing/EditCom...
File Source/core/editing/EditCommand.h (left):

https://codereview.chromium.org/1218293014/diff/1/Source/core/editing/EditCom...
Source/core/editing/EditCommand.h:87: virtual bool isSimpleEditCommand() const
override final { return true; }
It is interesting that compiler allows to have both |override| and |final|.

[yosin_UTC9, 2015-07-07 06:40:39]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-07-07 06:40:49]

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

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

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-07 06:55:06]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/6...)

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

The CQ bit was checked by tkent@chromium.org

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

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

[commit-bot: I haz the power, 2015-07-07 08:09:37]

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