Update histograms.xml for recent BadMessage values.

BUG=486916
TEST=Histogram properly labeled

Review URL: https://codereview.chromium.org/1128993005

Cr-Commit-Position: refs/heads/master@{#329439}

content/browser/bad_message.h

@@ -18,28 +18,28 @@ namespace bad_message {
 //
 // NOTE: Do not remove or reorder elements in this list. Add new entries at the
 // end. Items may be renamed but do not change the values. We rely on the enum
-// values in histograms.
+// values in histograms. Also update histograms.xml with any new values.
 enum BadMessageReason {
   NC_IN_PAGE_NAVIGATION = 0,
-  RFH_CAN_COMMIT_URL_BLOCKED,
-  RFH_CAN_ACCESS_FILES_OF_PAGE_STATE,
-  RFH_SANDBOX_FLAGS,
-  RFH_NO_PROXY_TO_PARENT,
-  RPH_DESERIALIZATION_FAILED,
-  RVH_CAN_ACCESS_FILES_OF_PAGE_STATE,
-  RVH_FILE_CHOOSER_PATH,
-  RWH_SYNTHETIC_GESTURE,
-  RWH_FOCUS,
-  RWH_BLUR,
-  RWH_SHARED_BITMAP,
-  RWH_BAD_ACK_MESSAGE,
-  RWHVA_SHARED_MEMORY,
-  SERVICE_WORKER_BAD_URL,
-  WC_INVALID_FRAME_SOURCE,
-  RWHVM_UNEXPECTED_FRAME_TYPE,
-  RFPH_DETACH,
-  DFH_BAD_EMBEDDER_MESSAGE,
-  NC_AUTO_SUBFRAME,
+  RFH_CAN_COMMIT_URL_BLOCKED = 1,
+  RFH_CAN_ACCESS_FILES_OF_PAGE_STATE = 2,
+  RFH_SANDBOX_FLAGS = 3,
+  RFH_NO_PROXY_TO_PARENT = 4,
+  RPH_DESERIALIZATION_FAILED = 5,
+  RVH_CAN_ACCESS_FILES_OF_PAGE_STATE = 6,
+  RVH_FILE_CHOOSER_PATH = 7,
+  RWH_SYNTHETIC_GESTURE = 8,
+  RWH_FOCUS = 9,
+  RWH_BLUR = 10,
+  RWH_SHARED_BITMAP = 11,
+  RWH_BAD_ACK_MESSAGE = 12,
+  RWHVA_SHARED_MEMORY = 13,
+  SERVICE_WORKER_BAD_URL = 14,
+  WC_INVALID_FRAME_SOURCE = 15,
+  RWHVM_UNEXPECTED_FRAME_TYPE = 16,
+  RFPH_DETACH = 17,
+  DFH_BAD_EMBEDDER_MESSAGE = 18,
+  NC_AUTO_SUBFRAME = 19,
   // Please add new elements here. The naming convention is abbreviated class
   // name (e.g. RenderFrameHost becomes RFH) plus a unique description of the
   // reason.

extensions/browser/bad_message.h

@@ -19,12 +19,12 @@ namespace bad_message {
 //
 // NOTE: Do not remove or reorder elements in this list. Add new entries at the
 // end. Items may be renamed but do not change the values. We rely on the enum
-// values in histograms.
+// values in histograms. Also update histograms.xml with any new values.
 enum BadMessageReason {
   EOG_BAD_ORIGIN = 0,
-  EVG_BAD_ORIGIN,
-  BH_BLOB_NOT_OWNED,
-  EH_BAD_EVENT_ID,
+  EVG_BAD_ORIGIN = 1,
+  BH_BLOB_NOT_OWNED = 2,
+  EH_BAD_EVENT_ID = 3,
   // Please add new elements here. The naming convention is abbreviated class
   // name (e.g. ExtensionHost becomes EH) plus a unique description of the
   // reason.

tools/metrics/histograms/histograms.xml

@@ -47602,7 +47602,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <int value="14" label="SERVICE_WORKER_BAD_URL"/>
   <int value="15" label="WC_INVALID_FRAME_SOURCE"/>
   <int value="16" label="RWHVM_UNEXPECTED_FRAME_TYPE"/>
-  <int value="17" label="DFH_BAD_EMBEDDER_MESSAGE"/>
+  <int value="17" label="RFPH_DETACH"/>
+  <int value="18" label="DFH_BAD_EMBEDDER_MESSAGE"/>
+  <int value="19" label="NC_AUTO_SUBFRAME"/>
 </enum>
 
 <enum name="BadMessageReasonExtensions" type="int">