• skym's avatar
    [Sync] Stop updating local device info time stamp on merge. · 459c7c57
    skym authored
    When merging, if we're given specifics that correspond to the local
    guid, we try to access specifics object for the local guid in
    |all_data_|. The problem with this, is that there isn't always an entry
    there if reconcile has not happened yet. This only happens when the
    initial sync round trip finishes before metadata loads from disk, but
    this race is happening to users, see the bug.
    
    The fix could be to check that |all_data_| actually contains an entry
    for the local guid, but it really doesn't matter. Local data completely
    cleared when sync is disabled, so reconcile will always create a local
    entry with a fresh time stamp around the time merge is called. There's
    no reason to complicate the logic inside of merge for no real benefit,
    so I've simply removed the set_last_updated_timestamp call.
    
    BUG=668938
    
    Review-Url: https://codereview.chromium.org/2536043002
    Cr-Commit-Position: refs/heads/master@{#435011}
    459c7c57
device_info_sync_bridge.cc 16 KB