Fix virtual/override/final usage in Source/core/{fetch,loader,streams,xmlhttprequest}/.

Fix virtual/override/final usage in Source/core/{fetch,loader,streams,xmlhttprequest}/.

- 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.
- Also replace { } with {} for updated lines to align with Chromium style.

This CL has no behavior changes.

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

[tkent, 2015-07-09 05:03:11]

Patchset #1 (id:1) has been deleted

[tkent, 2015-07-09 05:04:19]

tkent@chromium.org changed reviewers:
+ tyoshino@chromium.org, yhirano@chromium.org

[tkent, 2015-07-09 05:04:20]

tyoshino or yhirano, would you review this trivial change please?

[yhirano, 2015-07-09 05:11:55]

On 2015/07/09 05:04:20, tkent wrote:
> tyoshino or yhirano, would you review this trivial change please?

Sure.

[yhirano, 2015-07-09 05:36:38]

You replace some "{ }" with "{}", but some remain unchanged (e.g.
Source/core/fetch/CSSStyleSheetResource.h:67). Is this intentional?

[tkent, 2015-07-09 06:14:39]

On 2015/07/09 05:36:38, yhirano wrote:
> You replace some "{ }" with "{}", but some remain unchanged (e.g.
> Source/core/fetch/CSSStyleSheetResource.h:67). Is this intentional?

Yes.  I don't intend to replace all of "{ }" in files touched by this CL.

[tyoshino (SeeGerritForStatus), 2015-07-09 06:25:01]

lgtm

[yhirano, 2015-07-09 06:47:11]

lgtm

[tkent, 2015-07-09 07:24:13]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-07-09 07:24:45]

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

[commit-bot: I haz the power, 2015-07-09 07:29:17]

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