Commit 93db1962 authored by Mikel Astiz's avatar Mikel Astiz Committed by Commit Bot

Reassign TODOs related to SESSIONS cleanup

A newly filed bug will be used to track code cleanup.

Bug: 895455
Change-Id: If38fcc9311b441e68b43970a0baee8375750e3cd
Reviewed-on: https://chromium-review.googlesource.com/c/1280673Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599660}
parent b65449a2
...@@ -63,7 +63,7 @@ class SyncApiComponentFactory { ...@@ -63,7 +63,7 @@ class SyncApiComponentFactory {
// Creates and returns enabled datatypes and their controllers. // Creates and returns enabled datatypes and their controllers.
// |disabled_types| allows callers to prevent certain types from being // |disabled_types| allows callers to prevent certain types from being
// created (e.g. to honor command-line flags). // created (e.g. to honor command-line flags).
// TODO(crbug.com/681921): Remove |local_device_info_provider| once the // TODO(crbug.com/895455): Remove |local_device_info_provider| once the
// migration to USS is completed. // migration to USS is completed.
virtual DataTypeController::TypeVector CreateCommonDataTypeControllers( virtual DataTypeController::TypeVector CreateCommonDataTypeControllers(
ModelTypeSet disabled_types, ModelTypeSet disabled_types,
......
...@@ -80,7 +80,7 @@ class SyncClient { ...@@ -80,7 +80,7 @@ class SyncClient {
virtual bool HasPasswordStore() = 0; virtual bool HasPasswordStore() = 0;
// Returns a vector with all supported datatypes and their controllers. // Returns a vector with all supported datatypes and their controllers.
// TODO(crbug.com/681921): Remove |local_device_info_provider| once the // TODO(crbug.com/895455): Remove |local_device_info_provider| once the
// migration to USS is completed. // migration to USS is completed.
virtual DataTypeController::TypeVector CreateDataTypeControllers( virtual DataTypeController::TypeVector CreateDataTypeControllers(
LocalDeviceInfoProvider* local_device_info_provider) = 0; LocalDeviceInfoProvider* local_device_info_provider) = 0;
......
...@@ -112,7 +112,7 @@ class SyncData { ...@@ -112,7 +112,7 @@ class SyncData {
int64_t id_; int64_t id_;
// This may be null if the SyncData represents a deleted item. // This may be null if the SyncData represents a deleted item.
// TODO(crbug.com/681921): Remove when directory-based sessions sync is // TODO(crbug.com/895455): Remove when directory-based sessions sync is
// removed, the only remaining reader. // removed, the only remaining reader.
base::Time remote_modification_time_; base::Time remote_modification_time_;
...@@ -158,7 +158,7 @@ class SyncDataRemote : public SyncData { ...@@ -158,7 +158,7 @@ class SyncDataRemote : public SyncData {
// Return the last motification time according to the server. This may be null // Return the last motification time according to the server. This may be null
// if the SyncData represents a deleted item. // if the SyncData represents a deleted item.
// TODO(crbug.com/681921): Remove when directory-based sessions sync is // TODO(crbug.com/895455): Remove when directory-based sessions sync is
// removed, the only remaining reader. // removed, the only remaining reader.
const base::Time& GetModifiedTime() const; const base::Time& GetModifiedTime() const;
......
...@@ -18,7 +18,7 @@ class FaviconCache; ...@@ -18,7 +18,7 @@ class FaviconCache;
class OpenTabsUIDelegate; class OpenTabsUIDelegate;
class SessionsGlobalIdMapper; class SessionsGlobalIdMapper;
// TODO(crbug.com/681921): Remove this interface once the migration to USS // TODO(crbug.com/895455): Remove this interface once the migration to USS
// is completed and the old code removed. // is completed and the old code removed.
class AbstractSessionsSyncManager { class AbstractSessionsSyncManager {
public: public:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment