• Mikel Astiz's avatar
    Deflake and reenable bookmark sync tests · cfd41acf
    Mikel Astiz authored
    GUID-related tests have been flaky since introduction and recent
    investigations suggest that the underlying issue is FakeServer's lack of
    sending sync invalidations for certain APIs like InjectEntity().
    
    This patch modifies FakeServer to actually issue invalidations and
    removes some now redundant (and undesired) calls to
    TriggerSyncForModelTypes().
    
    Assumes this fixes the flakiness, all tests are reenabled.
    
    Bug: 1014086
    Change-Id: Ie0f597c024646586b9ec6c3dfbb761a86dadaad2
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879293Reviewed-by: default avatarMaksim Moskvitin <mmoskvitin@google.com>
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#710271}
    cfd41acf
fake_server.cc 18.9 KB