Fix problem where field trials using kExpirationYearInFuture could get disabled.

base/metrics/field_trial.h

Index: base/metrics/field_trial.h
===================================================================
--- base/metrics/field_trial.h	(revision 175535)
+++ base/metrics/field_trial.h	(working copy)
@@ -283,8 +283,9 @@
   // second process to mimic our state (i.e., provide the same group name).
   static const char kPersistentStringSeparator;  // Currently a slash.
 
-  // Define expiration year in future. It is initialized to two years from Now.
-  static int kExpirationYearInFuture;
+  // Year that is guaranteed to not be expired when instantiating a field trial
+  // via |FactoryGetFieldTrial()|. Set to two years from the build date.

[Ilya Sherman, 2013/01/08 21:18:53]

nit: Please use two spaces after the period for consistency with the surrounding
code.

[Alexei Svitkine (slow), 2013/01/09 15:24:58]

Done.

+  static int kExpirationYearNotExpired;

[Ilya Sherman, 2013/01/08 21:18:53]

nit: I think the previous name was slightly clearer for casual users of the API

[Alexei Svitkine (slow), 2013/01/08 21:56:22]

The reason I changed the name is because after this change, the date may not
necessarily be in the future. (e.g. if it's an old build and the build date is
over two years old, then this date may be in the past). So I chose a name that
communicates exactly what it's guaranteeing (i.e. the trial won't be expired),
rather than talking about "future", which is practically not relevant to users.

Let me know if you still want me to change it (and if so to what), or if this is
okay given my explanation.

[Ilya Sherman, 2013/01/08 22:02:04]

I agree that the new name is more technically correct, but I think that most
clients would rather have the name be clearer in its intent, even if it might be
strictly false.  IMO kExpirationYearNotExpired is a little hard to parse.  Some
other potential names: kNoExpirationYear, kNeverExpireYear, kUnexpiredYear,
kExpirationYearNeverExpire (though this last one has a similar issue w.r.t.
being kind of hard to parse).  WDYT?

[Alexei Svitkine (slow), 2013/01/09 15:24:58]

Changed to kNoExpirationYear.

 
   // Observer is notified when a FieldTrial's group is selected.
   class BASE_EXPORT Observer {