Java wrapper for DistilledPagePrefs.

components/dom_distiller/core/dom_distiller_service_android.h

+// Copyright 2014 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 COMPONENTS_DOM_DISTILLER_CORE_DOM_DISTILLER_SERVICE_ANDROID_H_
+#define COMPONENTS_DOM_DISTILLER_CORE_DOM_DISTILLER_SERVICE_ANDROID_H_
+
+#include <jni.h>
+
+#include "base/android/scoped_java_ref.h"
+

[nyquist, 2014/07/08 01:02:35]

Remove empty line.

[sunangel, 2014/07/08 20:41:00]

Done.

+#include "components/dom_distiller/core/dom_distiller_service.h"
+
+namespace dom_distiller{
+namespace android{
+
+class DomDistillerServiceFactoryAndroid;
+
+class DomDistillerServiceAndroid {

[nyquist, 2014/07/08 01:02:35]

What is this class? (add comment to explain).

[sunangel, 2014/07/08 20:41:00]

Done.

+
+ public:
+  friend class DomDistillerServiceFactoryAndroid;

[nyquist, 2014/07/08 01:02:35]

Add a comment which explains why this is needed.

[sunangel, 2014/07/08 20:41:00]

Done.

+  DomDistillerServiceAndroid(dom_distiller::DomDistillerService* ptr);

[nyquist, 2014/07/08 01:02:34]

s/ptr/service/. Also, remove namespace, since you're already in it I believe.

[sunangel, 2014/07/08 20:41:00]

Done.

+  static bool Register(JNIEnv* env);
+  virtual ~DomDistillerServiceAndroid();

[nyquist, 2014/07/08 01:02:35]

move up. see
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Declar...

[sunangel, 2014/07/08 20:41:00]

Done.

+  jlong GetDistilledPagePrefsPointer(JNIEnv* env, jobject obj);
+
+ private:
+  base::android::ScopedJavaGlobalRef<jobject> java_ref_;
+  dom_distiller::DomDistillerService* service_;
+};
+
+} // namespace android
+} // namespace dom_distiller
+
+#endif  // COMPONENTS_DOM_DISTILLER_CORE_DOM_DISTILLER_SERVICE_ANDROID_H