IPH - connect data saver previews

components/feature_engagement_tracker/public/android/java/src/org/chromium/components/feature_engagement_tracker/EventConstants.java

Index: components/feature_engagement_tracker/public/android/java/src/org/chromium/components/feature_engagement_tracker/EventConstants.java
diff --git a/components/feature_engagement_tracker/public/android/java/src/org/chromium/components/feature_engagement_tracker/EventConstants.java b/components/feature_engagement_tracker/public/android/java/src/org/chromium/components/feature_engagement_tracker/EventConstants.java
new file mode 100644
index 0000000000000000000000000000000000000000..1a741b42e0a64071793be016e5d5779e7c383840
--- /dev/null
+++ b/components/feature_engagement_tracker/public/android/java/src/org/chromium/components/feature_engagement_tracker/EventConstants.java
@@ -0,0 +1,18 @@
+// Copyright 2017 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.
+
+package org.chromium.components.feature_engagement_tracker;
+
+/**
+ * EventConstants contains the events declared in
+ * //components/feature_engagement_tracker/public/event_constants.h.
+ */
+public final class EventConstants {
+    public static final String DATA_SAVER_PREVIEW_OPENED = "data_saver_preview_opened";

[nyquist, 2017/04/18 23:09:04]

Since these are defined directly in the public subdir for Java, should we move
the .cc files for the constants for C++ to public as well?

[David Trainor- moved to gerrit, 2017/04/25 05:01:05]

I'm fine doing that.  We can move them over along with FeatureConstants in one
CL (or if Shakti's CL does it I'll wait for his to land).

+
+    /**
+     * Do not instantiate.
+     */
+    private EventConstants() {}
+}