Use DEFINE_TRACE in FrameSelection instead of DEFINE_VIRTUAL_TRACE

Use DEFINE_TRACE in FrameSelection instead of DEFINE_VIRTUAL_TRACE

This patch changes |FrameSelection| class to use |DEFINE_TRACE()| instead of
|DEFINE_VIRTUAL_TRACE()| since |FrameSelection| is final.

|DEFINE_VIRTUAL_TRACE()| was introduced by [1] as result of mechanical replacing
of |virtual void trace()|.

[1] https://chromium.googlesource.com/chromium/src/+/0ae5a4a2770e78d7cdc0025f1ebeebd797691b16

BUG=n/a
TEST=n/a; no behavior changes.

Committed: https://crrev.com/b80ad15447c25b8af821a851961ce078d9c2d042
Cr-Commit-Position: refs/heads/master@{#427030}

[yosin_UTC9, 2016-10-24 08:05:34]

Description was changed from

==========
2016-10-24T17:03:53

BUG=
==========

to

==========
2016-10-24T17:03:53

BUG=n/a
TEST=n/a; no behavior changes.
==========

[yosin_UTC9, 2016-10-24 08:05:42]

The CQ bit was checked by yosin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-24 08:05:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yosin_UTC9, 2016-10-24 08:20:24]

Description was changed from

==========
2016-10-24T17:03:53

BUG=n/a
TEST=n/a; no behavior changes.
==========

to

==========
Use DEFINE_TRACE in FrameSelection instead of DEFINE_VIRTUAL_TRACE

This patch changes |FrameSelection| class to use |DEFINE_TRACE()| instead of
|DEFINE_VIRTUAL_TRACE()| since |FrameSelection| is final.

|DEFINE_VIRTUAL_TRACE()| was introduced by [1] as result of mechanical replacing
of |virtual void trace()|.

[1]
https://chromium.googlesource.com/chromium/src/+/0ae5a4a2770e78d7cdc0025f1ebe...

BUG=n/a
TEST=n/a; no behavior changes.
==========

[yosin_UTC9, 2016-10-24 08:20:34]

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

[yosin_UTC9, 2016-10-24 08:20:34]

PTAL

[tkent, 2016-10-24 08:23:22]

lgtm.

Probably it is virtual because FrameSelection inherited from CaretBase.

[yosin_UTC9, 2016-10-24 08:24:20]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2016-10-24 08:24:22]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-10-24 08:24:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 09:21:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-24 09:23:58]

Description was changed from

==========
Use DEFINE_TRACE in FrameSelection instead of DEFINE_VIRTUAL_TRACE

This patch changes |FrameSelection| class to use |DEFINE_TRACE()| instead of
|DEFINE_VIRTUAL_TRACE()| since |FrameSelection| is final.

|DEFINE_VIRTUAL_TRACE()| was introduced by [1] as result of mechanical replacing
of |virtual void trace()|.

[1]
https://chromium.googlesource.com/chromium/src/+/0ae5a4a2770e78d7cdc0025f1ebe...

BUG=n/a
TEST=n/a; no behavior changes.
==========

to

==========
Use DEFINE_TRACE in FrameSelection instead of DEFINE_VIRTUAL_TRACE

This patch changes |FrameSelection| class to use |DEFINE_TRACE()| instead of
|DEFINE_VIRTUAL_TRACE()| since |FrameSelection| is final.

|DEFINE_VIRTUAL_TRACE()| was introduced by [1] as result of mechanical replacing
of |virtual void trace()|.

[1]
https://chromium.googlesource.com/chromium/src/+/0ae5a4a2770e78d7cdc0025f1ebe...

BUG=n/a
TEST=n/a; no behavior changes.

Committed: https://crrev.com/b80ad15447c25b8af821a851961ce078d9c2d042
Cr-Commit-Position: refs/heads/master@{#427030}
==========

[commit-bot: I haz the power, 2016-10-24 09:23:59]

Patchset 1 (id:??) landed as
https://crrev.com/b80ad15447c25b8af821a851961ce078d9c2d042
Cr-Commit-Position: refs/heads/master@{#427030}