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

Issue 862186 link

Starred by 1 user

Issue metadata

Status: Fixed
Closed: Jul 17
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task

issue 851433

Sign in to add a comment

Split ConsentAuditor interface from implementation.

Project Member Reported by, Jul 10

Issue description

This is needed to unlink FakeConsentAuditor from the implementation, which blocks enabling separate consents datatype by default.
Project Member

Comment 1 by, Jul 12

The following revision refers to this bug:

commit 7782405912e05feca6e0cf3808ab8f3c8ff70f2f
Author: vitaliii <>
Date: Thu Jul 12 15:40:45 2018

[Sync::Consent] Split ConsentAuditor interface from implementation.

Also rewrite FakeConsentAuditor to implement the interface instead of
inheriting from the implementation. This unblocks enabling the separate
datatype feature by default.

Bug:  862186 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I474f74f12278221ef6278c120576f1ff1a224c4a
Reviewed-by: Markus Heintz <>
Reviewed-by: Mihai Sardarescu <>
Commit-Queue: vitaliii <>
Cr-Commit-Position: refs/heads/master@{#574569}

Status: Fixed (was: Started)

Sign in to add a comment