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

Issue 612666 link

Starred by 1 user
Status: Fixed
Owner:
█ jwd@chromium.org
Last visit > 30 days ago
Closed: May 2016
Cc:
█ zkoch@chromium.org
gayane@chromium.org
asvitk...@chromium.org
skare@chromium.org
isherman@chromium.org
█ rkaplow@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature



Sign in to add a comment

Use low entropy for studies that send experiment IDs to Google.

Project Member Reported by jwd@chromium.org, May 18 2016 
Use low entropy for studies that send experiment IDs to Google.

Any study created by the variations service that sends experiment IDs to any Google property (web properties, updater, or Chrome sync) should always use low entropy when picking groups.
Project Member

Comment 1 by bugdroid1@chromium.org, May 18 2016 

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

commit 67c08f756c5907a7ff87fc59d2c6af947987881d
Author: jwd <jwd@chromium.org>
Date: Wed May 18 21:04:59 2016

Use low entropy for studies that send experiment IDs to Google properties.

This applies to all clients, even those that have metrics reporting enabled. This will cause a reshuffle of any study that is sending experiment IDs.

BUG= 612666 

Review-Url: https://codereview.chromium.org/1984003002
Cr-Commit-Position: refs/heads/master@{#394544}

[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/base/metrics/field_trial.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/base/metrics/field_trial.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/base/test/mock_entropy_provider.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/base/test/mock_entropy_provider.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/metrics/metrics_service.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/metrics/metrics_state_manager.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/metrics/metrics_state_manager.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/metrics/metrics_state_manager_unittest.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/entropy_provider_unittest.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/service/variations_service.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/service/variations_service.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/service/variations_service_unittest.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_processor.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_processor.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_processor_unittest.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_simulator.cc
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_simulator.h
[modify] https://crrev.com/67c08f756c5907a7ff87fc59d2c6af947987881d/components/variations/variations_seed_simulator_unittest.cc

Comment 2 by jwd@chromium.org, May 18 2016

Status: Fixed (was: Started)
►

Sign in to add a comment