Observer Support for DistilledPagePrefs

components/dom_distiller/core/distilled_page_prefs_android.cc

Index: components/dom_distiller/core/distilled_page_prefs_android.cc
diff --git a/components/dom_distiller/core/distilled_page_prefs_android.cc b/components/dom_distiller/core/distilled_page_prefs_android.cc
index a4edfbd690516145a0f87b2022e57c9c06627dc3..61891fbce42f90f6afe0a62960052f71a02635a2 100644
--- a/components/dom_distiller/core/distilled_page_prefs_android.cc
+++ b/components/dom_distiller/core/distilled_page_prefs_android.cc
@@ -5,10 +5,13 @@
 #include "components/dom_distiller/core/distilled_page_prefs_android.h"
 
 #include "components/dom_distiller/core/distilled_page_prefs.h"
+#include "components/dom_distiller/core/distilled_page_prefs_observer_wrapper.h"
 #include "components/dom_distiller/core/dom_distiller_service.h"
+

[nyquist, 2014/07/22 22:51:37]

Nit: Remove empty newline.

[sunangel, 2014/07/23 16:22:04]

Done.

 #include "jni/DistilledPagePrefs_jni.h"
 
 namespace dom_distiller {
+
 namespace android {
 
 DistilledPagePrefsAndroid::DistilledPagePrefsAndroid(
@@ -41,5 +44,22 @@ bool DistilledPagePrefsAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
+void DistilledPagePrefsAndroid::RemoveObserver(JNIEnv* env,

[nyquist, 2014/07/22 22:51:37]

Could you re-order these two functions to match the header?

[sunangel, 2014/07/23 16:22:04]

Done.

+                                               jobject obj,
+                                               jlong observerPtr) {
+  DistilledPagePrefsObserverWrapper* distilledPagePrefsObserverWrapper =
+      reinterpret_cast<DistilledPagePrefsObserverWrapper*>(observerPtr);
+  distilled_page_prefs_->RemoveObserver(distilledPagePrefsObserverWrapper);
+}
+
+void DistilledPagePrefsAndroid::AddObserver(JNIEnv* env,
+                                            jobject obj,
+                                            jlong observerPtr) {
+  DistilledPagePrefsObserverWrapper* distilledPagePrefsObserverWrapper =
+      reinterpret_cast<DistilledPagePrefsObserverWrapper*>(observerPtr);
+  distilled_page_prefs_->AddObserver(distilledPagePrefsObserverWrapper);
+}
+
 }  // namespace android
+
 }  // namespace dom_distiller