Expose the Site Engagement Service to Java.

chrome/browser/engagement/site_engagement_service_android.h

+// Copyright 2016 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.
+
+#ifndef CHROME_BROWSER_ENGAGEMENT_SITE_ENGAGEMENT_SERVICE_ANDROID_H_
+#define CHROME_BROWSER_ENGAGEMENT_SITE_ENGAGEMENT_SERVICE_ANDROID_H_
+
+#include <jni.h>
+
+#include "base/android/scoped_java_ref.h"
+#include "chrome/browser/engagement/site_engagement_service.h"
+
+// Wrapper class to expose the Site Engagement Service to Java. This class is
+// owned by the |service_| which it wraps, and is lazily created when

[gone, 2016/12/07 21:50:26]

when...?  Intrigued minds want to know.

[dominickn, 2016/12/08 01:03:54]

Oops.

+class SiteEngagementServiceAndroid {
+ public:
+  static bool Register(JNIEnv* env);
+
+  static SiteEngagementServiceAndroid* GetOrCreate(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& obj,
+      SiteEngagementService* service);
+
+  explicit SiteEngagementServiceAndroid(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& obj,
+      SiteEngagementService* service);

[gone, 2016/12/07 21:50:26]

nit: newline here

[dominickn, 2016/12/08 01:03:54]

Done.

+  ~SiteEngagementServiceAndroid();
+
+  double GetScore(JNIEnv* env,
+                  const base::android::JavaParamRef<jobject>& obj,
+                  const base::android::JavaParamRef<jstring>& jurl);
+
+  void ResetScoreForURL(JNIEnv* env,
+                        const base::android::JavaParamRef<jobject>& obj,
+                        const base::android::JavaParamRef<jstring>& jurl,
+                        double score);
+
+ private:
+  base::android::ScopedJavaGlobalRef<jobject> obj_;

[gone, 2016/12/07 21:50:26]

Use something more descriptive than obj?

[dominickn, 2016/12/08 01:03:54]

Done.

+  SiteEngagementService* service_;

[gone, 2016/12/07 21:50:26]

DISALLOW_COPY_ETC_ETC?

[dominickn, 2016/12/08 01:03:54]

Derp.

+};
+
+#endif  // CHROME_BROWSER_ENGAGEMENT_SITE_ENGAGEMENT_SERVICE_ANDROID_H_