Make DocumentMarker::AllMarkers() automatically track list of types

Make DocumentMarker::AllMarkers() automatically track list of types

Every time I try to add a new DocumentMarker, I get tripped up by this list
needing to be updated. If we write it like this, it will automatically start
including the new type, without any performance penalty.

BUG=707867
Review-Url: https://codereview.chromium.org/2857483002
Cr-Commit-Position: refs/heads/master@{#470029}
Committed: https://chromium.googlesource.com/chromium/src/+/b1bdbd3e0b51de1e6327e71d27e8b7425d475e04

[rlanday, 2017-05-02 02:14:34]

rlanday@chromium.org changed reviewers:
+ xiaochengh@chromium.org, yosin@chromium.org

[Xiaocheng, 2017-05-02 18:07:18]

lgtm

All reviewers based in Japan are OOO... Please find someone else as owner.

[rlanday, 2017-05-02 21:28:48]

rlanday@chromium.org changed reviewers:
+ ojan@chromium.org

[rlanday, 2017-05-02 21:28:48]

rlanday@chromium.org changed required reviewers:
+ ojan@chromium.org

[yosin_UTC9, 2017-05-08 03:41:55]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2017-05-08 03:41:56]

lgtm

[commit-bot: I haz the power, 2017-05-08 03:42:16]

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, 2017-05-08 03:42:16]

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

[commit-bot: I haz the power, 2017-05-08 03:42:17]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[rlanday, 2017-05-08 15:12:19]

rlanday@chromium.org changed required reviewers:
- ojan@chromium.org

[rlanday, 2017-05-08 15:12:23]

The CQ bit was checked by rlanday@chromium.org

[commit-bot: I haz the power, 2017-05-08 15:12:44]

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, 2017-05-08 17:22:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494256343841330, "parent_rev":
"37c0b451704be2e21c996595185d5bb520d3cf60", "commit_rev":
"b1bdbd3e0b51de1e6327e71d27e8b7425d475e04"}

[commit-bot: I haz the power, 2017-05-08 17:22:22]

Description was changed from

==========
Make DocumentMarker::AllMarkers() automatically track list of types

Every time I try to add a new DocumentMarker, I get tripped up by this list
needing to be updated. If we write it like this, it will automatically start
including the new type, without any performance penalty.

BUG=707867
==========

to

==========
Make DocumentMarker::AllMarkers() automatically track list of types

Every time I try to add a new DocumentMarker, I get tripped up by this list
needing to be updated. If we write it like this, it will automatically start
including the new type, without any performance penalty.

BUG=707867

Review-Url: https://codereview.chromium.org/2857483002
Cr-Commit-Position: refs/heads/master@{#470029}
Committed:
https://chromium.googlesource.com/chromium/src/+/b1bdbd3e0b51de1e6327e71d27e8...
==========

[commit-bot: I haz the power, 2017-05-08 17:22:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b1bdbd3e0b51de1e6327e71d27e8...