[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStores

[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275
This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.

Review-Url: https://codereview.chromium.org/2841653003
Cr-Commit-Position: refs/heads/master@{#467193}
Committed: https://chromium.googlesource.com/chromium/src/+/e92abbd14ad37670120e1c38ecf2cf6ce1a7eecd

[Gang Wu, 2017-04-24 18:28:54]

Description was changed from

==========
TypedURLSyncMetadataDatabase inherit SyncMetadataStore

BUG=708275
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherit SyncMetadataStore

BUG=708275
==========

[Gang Wu, 2017-04-24 18:28:54]

gangwu@chromium.org changed reviewers:
+ pavely@chromium.org

[Gang Wu, 2017-04-24 18:29:33]

PTAL

[Gang Wu, 2017-04-24 18:29:51]

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

[commit-bot: I haz the power, 2017-04-24 18:30:51]

Dry run: 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-04-24 18:42:10]

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

[commit-bot: I haz the power, 2017-04-24 18:42:11]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Gang Wu, 2017-04-24 18:44:35]

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

[Gang Wu, 2017-04-24 18:44:46]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-04-24 18:45:42]

Dry run: 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-04-24 20:27:29]

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

[commit-bot: I haz the power, 2017-04-24 20:27:30]

Dry run: This issue passed the CQ dry run.

[Gang Wu, 2017-04-24 21:47:09]

Description was changed from

==========
[USS] TypedURLSyncMetadataDatabase inherit SyncMetadataStore

BUG=708275
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275
==========

[pavely, 2017-04-25 17:23:53]

lgtm.
Could you add description explaining that this change is a step towards making
TypedUrlSyncBridge use SyncMetadataStoreChangeList.
http://go/cl-descriptions

[Gang Wu, 2017-04-25 17:28:17]

Description was changed from

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
==========

[Gang Wu, 2017-04-25 17:29:16]

Description was changed from

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.
==========

[Gang Wu, 2017-04-25 17:29:58]

Description was changed from

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.
==========

[Gang Wu, 2017-04-25 17:30:43]

gangwu@chromium.org changed reviewers:
+ brettw@chromium.org

[Gang Wu, 2017-04-25 17:31:09]

Hi Brett, PTAL

[brettw, 2017-04-25 22:40:53]

lgtm

[Gang Wu, 2017-04-25 23:47:38]

The CQ bit was checked by gangwu@chromium.org

[commit-bot: I haz the power, 2017-04-25 23:48:22]

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-04-26 01:18:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493164058579300,
"parent_rev": "d7d447a09a95eb7ad399eb8512f677a66e127d69", "commit_rev":
"e92abbd14ad37670120e1c38ecf2cf6ce1a7eecd"}

[commit-bot: I haz the power, 2017-04-26 01:20:08]

Description was changed from

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.
==========

to

==========
[USS] TypedURLSyncMetadataDatabase inherits SyncMetadataStore

BUG=708275

This is a follow up for CL https://codereview.chromium.org/2794413002/.
It allows typed urls to use common implementation of MetadataChangeList
(SyncMetadataStoreChangeList) instead of implementing its own.

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

[commit-bot: I haz the power, 2017-04-26 01:20:10]

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