GitLab

This is a reland of b9c8d354

CL was reverted because of test failures on many builders. The reason
for this failures is ModifyEntitySpecifics() being wrapped in DCHECK(),
which makes new test extremely flaky. Solution: unwrap it.

Original change's description:
> Add test for keystore key rotation
>
> New test mimics server-side behavior for triggering keystore key
> rotation: generation of new keystore key with touching the Nigori
> node. After that client should receive updated keystore keys and the
> same Nigori node as it already has (modulo metadata update). The client
> should commit the new Nigori node with rotated keystore key and this
> verified by checking |encryption_keybag| encryption key name.
>
> Bug: 922900
> Change-Id: I91b86bbb0f68a55fdcc10049fa90afa70dc32155
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887611
> Reviewed-by: Mikel Astiz <mastiz@chromium.org>
> Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
> Auto-Submit: Maksim Moskvitin <mmoskvitin@google.com>
> Cr-Commit-Position: refs/heads/master@{#712502}

Bug: 922900
Change-Id: I994af8befefb8207cedd37e53022329258455221
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899497
Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#712683}