tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index c7cf78af2c178bef8ab5982324d94b3e0b5bab5f..763c1d60759e39af74fba29a667520211143ae26 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -70310,6 +70310,7 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="-853594220" label="disable-new-avatar-menu"/>
   <int value="-836123854" label="wallet-service-use-sandbox"/>
   <int value="-820041355" label="enable-transition-compositing"/>
+  <int value="-814923711" label="herb-flavor"/>

[Ilya Sherman, 2016/02/03 01:13:21]

LGTM, though this is a ... really unclear flag name.  It's cute, but it will
make crash reports and whatnot that much harder to understand.  Just sayin'.

[gone, 2016/02/03 01:18:28]

Yeah, it's really meant to be an internal team-only flag for some
quick-and-dirty Android N UI prototyping.  It isn't really meant to be checked
for in crash reports.  The flag isn't going to live past M50/M51, but I can swap
it to something else if you prefer.

[Ilya Sherman, 2016/02/03 01:27:04]

Eh, I don't super mind either way.  But, all else equal, I do prefer clearer
flag names =)

[gone, 2016/02/03 01:57:28]

How's this new patch look?  Switched to "tab-management-experiment-type" for the
flag, but kept using the name Herb everywhere in the code because it's a
mouthful.

   <int value="-814097014" label="disable-session-crashed-bubble"/>
   <int value="-813474479" label="site-per-process"/>
   <int value="-802348444" label="disable-site-engagement-service"/>