Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 1009673002: Remove enhanced bookmarks sync experiment (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: . Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 7907011c76aee799a6b2c4be0c289222572fef6d..a7fa207749d66b8599ab1dd829696533842c6d8a 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -6801,17 +6801,6 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
-<histogram name="EnhancedBookmarks.SyncExperimentState"
- enum="BookmarksExperimentState">
Yaron 2015/03/16 19:41:20 You can remove the <enum> too
Mark P 2015/03/16 19:46:13 Do not remove a histogram declaration here. Inste
Mike Wittman 2015/03/16 20:15:48 Done.
- <owner>noyau@chromium.org</owner>
- <owner>yefim@chromium.org</owner>
- <summary>
- Captures the state the enhanced bookmark experiment is in. Recorded on
- startup. To be removed once the enhanced bookmark experiment is finished.
- see crbug/323423.
- </summary>
-</histogram>
-
<histogram name="Enterprise.AutoEnrollmentExtraTime" units="milliseconds">
<owner>mnissler@chromium.org</owner>
<summary>
@@ -52818,7 +52807,6 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="824961931" label="use-simple-cache-backend"/>
<int value="834326277" label="enable-answers-in-suggest"/>
<int value="835018878" label="disable-quic"/>
- <int value="838887742" label="manual-enhanced-bookmarks"/>
Mark P 2015/03/16 19:46:14 Please do not remove this or the enum below. It h
Mike Wittman 2015/03/16 20:15:48 Done.
<int value="851085848" label="enable-settings-window"/>
<int value="855746780" label="disable-physical-keyboard-autocorrect"/>
<int value="857445869" label="enable-captive-portal-bypass-proxy"/>
@@ -52846,7 +52834,6 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="1087235172" label="file-manager-enable-new-audio-player"/>
<int value="1090377940" label="enable-quic-https"/>
<int value="1095061640" label="enable-prominent-url-app-flow"/>
- <int value="1104948452" label="manual-enhanced-bookmarks-optout"/>
<int value="1105439588" label="enable-swipe-selection"/>
<int value="1107543566" label="enable-one-copy"/>
<int value="1108663108" label="disable-device-discovery-notifications"/>

Powered by Google App Engine
This is Rietveld 408576698