• Mikel Astiz's avatar
    Fix CHECK that relies on external sync sessions data · 64d2239b
    Mikel Astiz authored
    Tab sync entities are expected to have a valid |tab_id| field over the
    sync protocol, but clients should not crash if the received data is
    corrupt, e.g. committed by a buggy client.
    
    Prior to this patch, foreign sessions received in SessionsSyncManager,
    processed via UpdateTrackerWithSpecifics(), could trigger a CHECK
    failure in SyncedSessionTracker::GetTab().
    
    Bug: 840876
    Change-Id: Ie59cfb9c24934695a1b7b6187dbad48451f8c8ea
    Reviewed-on: https://chromium-review.googlesource.com/1051233Reviewed-by: default avatarJan Krcal <jkrcal@chromium.org>
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#557124}
    64d2239b
synced_session_tracker.cc 30.2 KB