Use new iOS NTP experiment ids.

chrome/common/metrics/variations/variation_ids.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_COMMON_METRICS_VARIATIONS_VARIATION_IDS_H_
 #define CHROME_COMMON_METRICS_VARIATIONS_VARIATION_IDS_H_
 
 namespace chrome_variations {
 
 // A list of Chrome Variation IDs. These IDs are associated with FieldTrials
@@ skipping 256 matching lines @@
   AUTOCOMPLETE_DYNAMIC_FIELD_TRIAL_RANGE2_ID_MIN = 3311947,
   AUTOCOMPLETE_DYANMIC_FIELD_TRIAL_RANGE2_ID_MAX = 3312046,
 
   // iOS Phone New NTP trial.
   // Range: 3312047 - 3312050 (Beta); 3312100 - 3312103 (Stable)
   //        3312112 - 3312113 (Stable)
   IOS_PHONE_NEW_NTP_OMNIBOX_HINT_BETA = 3312047,
   IOS_PHONE_NEW_NTP_CONTROL_1_BETA = 3312048,
   IOS_PHONE_NEW_NTP_FAKEBOX_HINT_BETA = 3312049,
   IOS_PHONE_NEW_NTP_CONTROL_2_BETA = 3312050,
   IOS_PHONE_NEW_NTP_OMNIBOX_HINT_STABLE = 3312100,

[rohitrao (ping after 24h), 2014/02/14 22:35:43]

Tablet is actually ok, I think.  It's the phone stable IDs that we'll need to
change.

I don't think we need to bother updating the "V3" IDs, since we'll just drop
those groups completely.

[justincohen, 2014/02/14 23:20:51]

I just removed V3.  Is that correct?

On 2014/02/14 22:35:43, rohitrao wrote:

   IOS_PHONE_NEW_NTP_CONTROL_1_STABLE = 3312101,
   IOS_PHONE_NEW_NTP_FAKEBOX_HINT_STABLE = 3312102,
   IOS_PHONE_NEW_NTP_CONTROL_2_STABLE = 3312103,
   IOS_PHONE_NEW_NTP_OMNIBOX_HINT_V3_STABLE = 3312112,
   IOS_PHONE_NEW_NTP_CONTROL_3_STABLE = 3312113,
 
   // iOS Tablet New NTP trial.
-  // Range: 3312104 - 3312107 (Beta); 3312108 - 3312111 (Stable)
+  // Range: 3312104 - 3312107 (Beta); 3312108 - 3312111 (Unused);

[rohitrao (ping after 24h), 2014/02/14 22:35:43]

It's probably worth using the deprecated/retired comment format from other IDs,
as in line 142-148.

[justincohen, 2014/02/14 23:20:51]

Done.

+  // 3312114 - 3312117 (Stable)
   IOS_TABLET_NEW_NTP_OMNIBOX_HINT_BETA = 3312104,
   IOS_TABLET_NEW_NTP_CONTROL_1_BETA = 3312105,
   IOS_TABLET_NEW_NTP_FAKEBOX_HINT_BETA = 3312106,
   IOS_TABLET_NEW_NTP_CONTROL_2_BETA = 3312107,
-  IOS_TABLET_NEW_NTP_OMNIBOX_HINT_STABLE = 3312108,
-  IOS_TABLET_NEW_NTP_CONTROL_1_STABLE = 3312109,
-  IOS_TABLET_NEW_NTP_FAKEBOX_HINT_STABLE = 3312110,
-  IOS_TABLET_NEW_NTP_CONTROL_2_STABLE = 3312111,
+  IOS_TABLET_NEW_NTP_OMNIBOX_HINT_STABLE_UNUSED = 3312108,

[rohitrao (ping after 24h), 2014/02/14 22:35:43]

Do you prefer _UNUSED, or the _MIN/_MAX format the suggest IDs use?

[justincohen, 2014/02/14 23:20:51]

I went with MIN/MAX.  I wasn't sure what to name it, tho, does
IOS_PHONE_NEW_NTP_2014_Q1_ID_MIN and IOS_PHONE_NEW_NTP_2014_Q1_ID2_MIN make
sense?

It seems a little odd.

On 2014/02/14 22:35:43, rohitrao wrote:

+  IOS_TABLET_NEW_NTP_CONTROL_1_STABLE_UNUSED = 3312109,
+  IOS_TABLET_NEW_NTP_FAKEBOX_HINT_STABLE_UNUSED = 3312110,
+  IOS_TABLET_NEW_NTP_CONTROL_2_STABLE_UNUSED = 3312111,
+  IOS_TABLET_NEW_NTP_OMNIBOX_HINT_STABLE = 3312114,
+  IOS_TABLET_NEW_NTP_CONTROL_1_STABLE = 3312115,
+  IOS_TABLET_NEW_NTP_FAKEBOX_HINT_STABLE = 3312116,
+  IOS_TABLET_NEW_NTP_CONTROL_2_STABLE = 3312117,
 
   // ExtensionInstallPrompt field trial.
   EXTENSION_INSTALL_PROMPT_EXPERIMENT_ID_MIN = 3312051,
   EXTENSION_INSTALL_PROMPT_EXPERIMENT_ID_MAX = 3312099,
 
   // NEXT ID: When adding new IDs, please add them above this section, starting
   // with the value of NEXT_ID, and updating NEXT_ID to (end of your reserved
   // range) + 1.
-  NEXT_ID = 3312114,
+  NEXT_ID = 3312118,
 
   // USABLE IDs END HERE.
   //
   // The largest possible Chrome variation ID in the reserved range. When
   // defining your variation IDs, DO NOT exceed this value - GWS will ignore
   // your experiment!
   MAXIMUM_ID = 3399999,
 };
 
 }  // namespace chrome_variations
 
 #endif  // CHROME_COMMON_METRICS_VARIATIONS_VARIATION_IDS_H_