Clean up virtual methods in MarkupAccumulator and subclasses

Clean up virtual methods in MarkupAccumulator and subclasses

Clean up virtual methods in MarkupAccumulator and subclasses:
- Use OVERRIDE for overriden virtual methods in StyledMarkupAccumulator and
  SerializerMarkupAccumulator
- Mark StyledMarkupAccumulator and SerializerMarkupAccumulator and FINAL as
  these are never subclassed and they override several virtual methods.
- MarkupAccumulator::appendString() does not need to be virtual as it does not
  need to be overriden

R=adamk

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

[adamk, 2014-01-07 19:10:47]

https://codereview.chromium.org/121043005/diff/1/Source/core/editing/markup.cpp
File Source/core/editing/markup.cpp (right):

https://codereview.chromium.org/121043005/diff/1/Source/core/editing/markup.c...
Source/core/editing/markup.cpp:121: void appendString(const String& s) { return
MarkupAccumulator::appendString(s); }
Why is this needed at all?

[Inactive, 2014-01-07 19:42:06]

https://codereview.chromium.org/121043005/diff/1/Source/core/editing/markup.cpp
File Source/core/editing/markup.cpp (right):

https://codereview.chromium.org/121043005/diff/1/Source/core/editing/markup.c...
Source/core/editing/markup.cpp:121: void appendString(const String& s) { return
MarkupAccumulator::appendString(s); }
On 2014/01/07 19:10:47, adamk wrote:
> Why is this needed at all?

To make it public. It is protected in the base class.

[adamk, 2014-01-07 19:47:23]

lgtm

[commit-bot: I haz the power, 2014-01-07 19:48:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/121043005/1

[commit-bot: I haz the power, 2014-01-07 21:31:14]

Change committed as 164614