Restrict transmission of external exp ids to signed in users.

components/variations/variations_associated_data.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/variations/variations_associated_data.h"
 
 #include <map>
 #include <utility>
 #include <vector>
 
@@ skipping 20 matching lines @@
     return base::Singleton<GroupMapAccessor>::get();
   }
 
   // Note that this normally only sets the ID for a group the first time, unless
   // |force| is set to true, in which case it will always override it.
   void AssociateID(IDCollectionKey key,
                    const ActiveGroupId& group_identifier,
                    const VariationID id,
                    const bool force) {
 #if !defined(NDEBUG)
-    DCHECK_EQ(3, ID_COLLECTION_COUNT);
-    // Ensure that at most one of the trigger/non-trigger web property IDs are
-    // set.
-    if (key == GOOGLE_WEB_PROPERTIES || key == GOOGLE_WEB_PROPERTIES_TRIGGER) {
-      IDCollectionKey other_key = key == GOOGLE_WEB_PROPERTIES ?
-          GOOGLE_WEB_PROPERTIES_TRIGGER : GOOGLE_WEB_PROPERTIES;
-      DCHECK_EQ(EMPTY_ID, GetID(other_key, group_identifier));
+    DCHECK_EQ(4, ID_COLLECTION_COUNT);
+    // Ensure that at most one of the trigger/non-trigger/signed-in web property
+    // IDs are set.
+    if (key == GOOGLE_WEB_PROPERTIES || key == GOOGLE_WEB_PROPERTIES_TRIGGER ||
+        key == GOOGLE_WEB_PROPERTIES_SIGNED_IN) {
+      if (key != GOOGLE_WEB_PROPERTIES)
+        DCHECK_EQ(EMPTY_ID, GetID(GOOGLE_WEB_PROPERTIES, group_identifier));
+      if (key != GOOGLE_WEB_PROPERTIES_TRIGGER) {
+        DCHECK_EQ(EMPTY_ID,
+                  GetID(GOOGLE_WEB_PROPERTIES_TRIGGER, group_identifier));
+      }
+      if (key != GOOGLE_WEB_PROPERTIES_SIGNED_IN) {
+        DCHECK_EQ(EMPTY_ID,
+                  GetID(GOOGLE_WEB_PROPERTIES_SIGNED_IN, group_identifier));
+      }
     }
 
     // Validate that all collections with this |group_identifier| have the same
     // associated ID.
     for (int i = 0; i < ID_COLLECTION_COUNT; ++i) {
       IDCollectionKey other_key = static_cast<IDCollectionKey>(i);
       if (other_key == key)
         continue;
       VariationID other_id = GetID(other_key, group_identifier);
       DCHECK(other_id == EMPTY_ID || other_id == id);
@@ skipping 140 matching lines @@
   GroupMapAccessor::GetInstance()->ClearAllMapsForTesting();
 }
 
 void ClearAllVariationParams() {
   base::FieldTrialParamAssociator::GetInstance()->ClearAllParamsForTesting();
 }
 
 }  // namespace testing
 
 }  // namespace variations