tools/metrics/histograms/enums.xml

Index: tools/metrics/histograms/enums.xml
diff --git a/tools/metrics/histograms/enums.xml b/tools/metrics/histograms/enums.xml
index 2da4e0a41bf2e818247a0cfc741e904cdd830830..5c7ef300e6a14d6c8e11806d30de8aa035713301 100644
--- a/tools/metrics/histograms/enums.xml
+++ b/tools/metrics/histograms/enums.xml
@@ -22315,7 +22315,6 @@ from previous Chrome versions.
   <int value="-1028733699" label="MacViewsWebUIDialogs:disabled"/>
   <int value="-1022971520" label="enable-search-button-in-omnibox-for-str"/>
   <int value="-1020450980" label="gesture-deletion"/>
-  <int value="-1016202433" label="disable-add-to-shelf"/>
   <int value="-1015006759" label="ImportantSitesInCBD:disabled"/>
   <int value="-998255750" label="ExperimentalKeyboardLockUI:enabled"/>
   <int value="-996673716" label="enable-web-app-frame"/>
@@ -22503,6 +22502,7 @@ from previous Chrome versions.
   <int value="-340023285" label="WebNFC:enabled"/>
   <int value="-329727402" label="disable-files-quick-view"/>
   <int value="-328361990" label="enable-experimental-extension-apis"/>
+  <int value="-326083626" label="AppBanners:disabled"/>
   <int value="-323831744" label="token-binding:enabled"/>
   <int value="-322937746" label="disable-desktop-capture-picker-new-ui"/>
   <int value="-322827131" label="tab-management-experiment-type-basil"/>
@@ -22682,7 +22682,6 @@ from previous Chrome versions.
   <int value="371268743" label="OmniboxUIExperimentVerticalMargin:disabled"/>
   <int value="372460068" label="QuickUnlockFingerprint:disabled"/>
   <int value="377093001" label="WebRtcHWH264Encoding:disabled"/>
-  <int value="379326303" label="enable-add-to-shelf"/>
   <int value="379428799" label="security-chip-animation"/>
   <int value="385969127" label="disable-win32k-lockdown"/>
   <int value="387178525" label="VideoFullscreenOrientationLock:enabled"/>
@@ -22847,6 +22846,7 @@ from previous Chrome versions.
   <int value="943319566" label="enable-intent-picker"/>
   <int value="952558794" label="enable-remote-assistance"/>
   <int value="980396200" label="enable-new-korean-ime"/>
+  <int value="981818901" label="AppBanners:enabled"/>
   <int value="982032277" label="NTPOfflineBadge:disabled"/>
   <int value="982511393" label="NTPArticleSuggestions:disabled"/>
   <int value="983311394" label="tab-management-experiment-type"/>
@@ -22965,6 +22965,7 @@ from previous Chrome versions.
   <int value="1320201920" label="enable-touchpad-three-finger-click"/>
   <int value="1330264457" label="OmniboxUIExperimentVerticalLayout:disabled"/>
   <int value="1344833841" label="ImeThread:enabled"/>
+  <int value="1350179532" label="enable-add-to-shelf (deprecated)"/>

[Matt Giuca, 2017/06/22 06:51:12]

Ugh if you were forced to change the value of the int then this isn't going to
have an effect (the whole point was to keep tracking the old name).

I'm not fussed but I want to do it properly. Let's ask someone from histograms
team whether to:
a) Just leave the old names as-is,
b) Delete them,
c) Something else.

[Ilya Sherman, 2017/06/22 16:29:28]

Typically we leave the old name as is.  Having the " (deprecated)" annotation is
kind of nice though; so if you feel motivated to fix the test to, say, ignore
such an annotation when computing the appropriate number, I'm on board with
that.

   <int value="1351830811" label="do-not-ignore-autocomplete-off"/>
   <int value="1352447982" label="enable-lcd-text"/>
   <int value="1359972809" label="enable-gesture-deletion"/>
@@ -23000,6 +23001,7 @@ from previous Chrome versions.
   <int value="1431050645" label="PayWithGoogleV1:disabled"/>
   <int value="1435018419" label="AutofillUpstreamRequestCvcIfMissing:disabled"/>
   <int value="1441897340" label="AndroidSpellCheckerNonLowEnd:enabled"/>
+  <int value="1442261297" label="disable-add-to-shelf (deprecated)"/>
   <int value="1442798825" label="enable-quic"/>
   <int value="1442830837" label="MemoryAblation:disabled"/>
   <int value="1454363479" label="disable-storage-manager"/>