Java wrapper for DistilledPagePrefs.

components/dom_distiller/core/reader_mode_preferences_android.cc

+// 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.
+
+#include "components/dom_distiller/core/reader_mode_preferences_android.h"
+#include "components/dom_distiller/core/reader_mode_preferences.h"
+
+#include "jni/ReaderModePreferencesAndroid_jni.h"
+
+namespace dom_distiller {
+namespace reader_mode_preferences {

[nyquist, 2014/06/30 20:33:24]

remove namespace

[sunangel, 2014/07/07 21:21:30]

Done.

+namespace android {
+
+void SetHighContrastMode(JNIEnv* env,

[nyquist, 2014/06/30 20:33:24]

This should probably be SetTheme

[sunangel, 2014/07/07 21:21:30]

Done.

+                         jobject obj,
+                         jlong ptr,
+                         jboolean enabled) {

[nyquist, 2014/06/30 20:33:24]

Shouldn't this take in a theme?
You might have to auto-generate the Java classes for this enum at that point.
See for example:
./components/cronet/android/java/src/org/chromium/net/UrlRequestError.template

[sunangel, 2014/07/07 21:21:30]

Done.

+  ReaderModePrefs* myReaderModePrefs = reinterpret_cast<ReaderModePrefs*>(ptr);
+  if (enabled) {
+    myReaderModePrefs->SetThemePref(ReaderModePrefs::Theme::kHighContrast);
+  } else {
+    myReaderModePrefs->SetThemePref(ReaderModePrefs::Theme::kNormal);
+  }
+}
+
+bool RegisterReaderModePrefsAndroid(JNIEnv* env) {

[nyquist, 2014/06/30 20:33:24]

DistilledPagePrefs

[sunangel, 2014/07/07 21:21:30]

just turned into register because made it in a class
On 2014/06/30 20:33:24, nyquist wrote:

+  return RegisterNativesImpl(env);
+}
+
+}  // namespace android
+}  // namespace reader_mode_preferences
+}  // namespace dom_distiller