New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 860434 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 9
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Task

Blocking:
issue 851433



Sign in to add a comment

SingleClient integration tests for USER_CONSENTS

Project Member Reported by vitaliii@chromium.org, Jul 5

Issue description

We have no browser tests for the new consents type.
We definitely would like some (even if just basic).
 
Labels: sync-fixit-2018q3
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2b1d6c5728954f8bebc208faa7558ab09bd9590c

commit 2b1d6c5728954f8bebc208faa7558ab09bd9590c
Author: vitaliii <vitaliii@chromium.org>
Date: Mon Jul 09 13:12:50 2018

[Sync::Consent] Add basic browser tests.

Add 2 simple browser tests for submitting user consents through a
separate datatype (in contrast to user events).
The approach is very similar to USER_EVENTS browser tests.

Bug:  860434 
Change-Id: I4e980e49d0490813143ad8aff15d308143019329
Reviewed-on: https://chromium-review.googlesource.com/1128865
Commit-Queue: vitaliii <vitaliii@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573283}
[add] https://crrev.com/2b1d6c5728954f8bebc208faa7558ab09bd9590c/chrome/browser/sync/test/integration/single_client_user_consents_sync_test.cc
[modify] https://crrev.com/2b1d6c5728954f8bebc208faa7558ab09bd9590c/chrome/test/BUILD.gn

Status: Fixed (was: Started)

Sign in to add a comment