Move PropertyRegistry::Registration into PropertyRegistration

Move PropertyRegistry::Registration into PropertyRegistration

Before this change PropertyRegistration represented the C++ implementation
of the PropertyRegistration IDL interface and nothing else.
This patch moves the Registration class out of PropertyRegistry and merges
it with PropertyRegistration.

This change is to enable forward declarations to what was
PropertyRegistry::Registration by making it a top level class.
C++ doesn't let you forward declare inner classes.

This change is required for removing a dependency on Document in:
https://codereview.chromium.org/2654783004

BUG=681424
Review-Url: https://codereview.chromium.org/2676993002
Cr-Commit-Position: refs/heads/master@{#448566}
Committed: https://chromium.googlesource.com/chromium/src/+/059255004a9e29dab9936f0527da655a148ce459

[alancutter (OOO until 2018), 2017-02-07 00:57:35]

alancutter@chromium.org changed reviewers:
+ sashab@chromium.org

[alancutter (OOO until 2018), 2017-02-07 01:03:11]

Description was changed from

==========
Move PropertyRegistry::Registration into PropertyRegistration

Before this change PropertyRegistration represented the C++ implementation
of the PropertyRegistration IDL interface and nothing else.
This patch moves the Registration class out of PropertyRegistry and merges
it with PropertyRegistration.

This change is to enable forward declarations to what was
PropertyRegistry::Registration by making it a top level class.
C++ doesn't let you forward declare inner classes.

BUG=681424
==========

to

==========
Move PropertyRegistry::Registration into PropertyRegistration

Before this change PropertyRegistration represented the C++ implementation
of the PropertyRegistration IDL interface and nothing else.
This patch moves the Registration class out of PropertyRegistry and merges
it with PropertyRegistration.

This change is to enable forward declarations to what was
PropertyRegistry::Registration by making it a top level class.
C++ doesn't let you forward declare inner classes.

This change is required for removing a dependency on Document in:
https://codereview.chromium.org/2654783004

BUG=681424
==========

[sashab, 2017-02-07 01:38:14]

Great cl description! :) LGTM

[alancutter (OOO until 2018), 2017-02-07 05:05:28]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2017-02-07 05:05:46]

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-02-07 06:51:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486443928759920,
"parent_rev": "14876d250a91001619418257e7f50d6b645e4588", "commit_rev":
"059255004a9e29dab9936f0527da655a148ce459"}

[commit-bot: I haz the power, 2017-02-07 06:52:41]

Description was changed from

==========
Move PropertyRegistry::Registration into PropertyRegistration

Before this change PropertyRegistration represented the C++ implementation
of the PropertyRegistration IDL interface and nothing else.
This patch moves the Registration class out of PropertyRegistry and merges
it with PropertyRegistration.

This change is to enable forward declarations to what was
PropertyRegistry::Registration by making it a top level class.
C++ doesn't let you forward declare inner classes.

This change is required for removing a dependency on Document in:
https://codereview.chromium.org/2654783004

BUG=681424
==========

to

==========
Move PropertyRegistry::Registration into PropertyRegistration

Before this change PropertyRegistration represented the C++ implementation
of the PropertyRegistration IDL interface and nothing else.
This patch moves the Registration class out of PropertyRegistry and merges
it with PropertyRegistration.

This change is to enable forward declarations to what was
PropertyRegistry::Registration by making it a top level class.
C++ doesn't let you forward declare inner classes.

This change is required for removing a dependency on Document in:
https://codereview.chromium.org/2654783004

BUG=681424

Review-Url: https://codereview.chromium.org/2676993002
Cr-Commit-Position: refs/heads/master@{#448566}
Committed:
https://chromium.googlesource.com/chromium/src/+/059255004a9e29dab9936f0527da...
==========

[commit-bot: I haz the power, 2017-02-07 06:52:42]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/059255004a9e29dab9936f0527da...