New issue
Advanced search Search tips

Issue 903276 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 18
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 801985



Sign in to add a comment

Disable invalidations for noisy datatypes on Clank.

Project Member Reported by melandory@chromium.org, Nov 8

Issue description

The Tango based invalidations infrastructure wasn't listening for the invalidations for the noisy datatypes [1] -> SESSIONS, FAVICON_TRACKING, FAVICON_IMAGES

The new implementation should follow the same pattern


[1] https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/invalidation/InvalidationController.java?l=185&rcl=90815d42b45e4dc346f85df41a55cbe2ff0b9b8a
 
Status: Available (was: Untriaged)
Blocking: 801985
Owner: mamir@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 28

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

commit 2e6a8ea798a0eec3e0c008832ae428ae7221e9ea
Author: Tanja Gornak <melandory@chromium.org>
Date: Wed Nov 28 14:40:59 2018

[Tango->FCM] Disable invalidations for noisy datatypes on Clank.

Bug:  903276 
Change-Id: Id679d9ae73535fa4f5f168c2745cdd65991c66cc
Reviewed-on: https://chromium-review.googlesource.com/c/1352311
Commit-Queue: Tatiana Gornak <melandory@chromium.org>
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611688}
[modify] https://crrev.com/2e6a8ea798a0eec3e0c008832ae428ae7221e9ea/components/sync/BUILD.gn
[modify] https://crrev.com/2e6a8ea798a0eec3e0c008832ae428ae7221e9ea/components/sync/driver/glue/sync_backend_host_impl.cc
[modify] https://crrev.com/2e6a8ea798a0eec3e0c008832ae428ae7221e9ea/components/sync/driver/glue/sync_backend_host_impl_unittest.cc

Comment 5 by melandory@google.com, Jan 18 (4 days ago)

Status: Fixed (was: Available)

Sign in to add a comment