General code cleanup of ChromeClient-related classes.

General code cleanup of ChromeClient-related classes.

- Code simplification
- Remove |virtual| of |override| functions
- Remove unnecessary includes and class declarations
- Use nullptr instead of 0
- Use range-based |for|
- etc.

This CL has no behavior changes.

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

[tkent, 2015-06-03 06:36:56]

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

[tkent, 2015-06-03 06:36:56]

yosin@, please review this.

[kouhei (in TOK), 2015-06-03 06:38:03]

kouhei@chromium.org changed reviewers:
+ kouhei@chromium.org

[kouhei (in TOK), 2015-06-03 06:38:03]

lgtm

[tkent, 2015-06-03 06:38:08]

tkent@chromium.org changed reviewers:
- kouhei@chromium.org

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

lgtm

https://codereview.chromium.org/1152413003/diff/1/Source/core/svg/graphics/SV...
File Source/core/svg/graphics/SVGImageChromeClient.h (right):

https://codereview.chromium.org/1152413003/diff/1/Source/core/svg/graphics/SV...
Source/core/svg/graphics/SVGImageChromeClient.h:43: bool
isSVGImageChromeClient() const override;
Q: Do you recommend |override| than |final| in |final| class?

[tkent, 2015-06-03 07:22:20]

https://codereview.chromium.org/1152413003/diff/1/Source/core/svg/graphics/SV...
File Source/core/svg/graphics/SVGImageChromeClient.h (right):

https://codereview.chromium.org/1152413003/diff/1/Source/core/svg/graphics/SV...
Source/core/svg/graphics/SVGImageChromeClient.h:43: bool
isSVGImageChromeClient() const override;
On 2015/06/03 07:15:27, Yosi_UTC9 wrote:
> Q: Do you recommend |override| than |final| in |final| class?

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

[tkent, 2015-06-03 08:06:36]

The CQ bit was checked by tkent@chromium.org

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

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

[commit-bot: I haz the power, 2015-06-03 09:08:35]

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

[commit-bot: I haz the power, 2015-06-03 09:08:36]

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-06-03 09:08:56]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-06-03 09:09:21]

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

[commit-bot: I haz the power, 2015-06-03 10:08:59]

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