Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Side by Side Diff: components/dom_distiller/core/distilled_page_prefs_android.cc

Issue 403323005: Observer Support for DistilledPagePrefs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added removeObserver after test is done, styling Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/dom_distiller/core/distilled_page_prefs_android.h" 5 #include "components/dom_distiller/core/distilled_page_prefs_android.h"
6 6
7 #include "components/dom_distiller/core/distilled_page_prefs.h" 7 #include "components/dom_distiller/core/distilled_page_prefs.h"
8 #include "components/dom_distiller/core/distilled_page_prefs_observer_wrapper.h"
8 #include "components/dom_distiller/core/dom_distiller_service.h" 9 #include "components/dom_distiller/core/dom_distiller_service.h"
10
9 #include "jni/DistilledPagePrefs_jni.h" 11 #include "jni/DistilledPagePrefs_jni.h"
10 12
11 namespace dom_distiller { 13 namespace dom_distiller {
14
12 namespace android { 15 namespace android {
13 16
14 DistilledPagePrefsAndroid::DistilledPagePrefsAndroid( 17 DistilledPagePrefsAndroid::DistilledPagePrefsAndroid(
15 JNIEnv* env, 18 JNIEnv* env,
16 jobject obj, 19 jobject obj,
17 DistilledPagePrefs* distillerPagePrefsPtr) 20 DistilledPagePrefs* distillerPagePrefsPtr)
18 : distilled_page_prefs_(distillerPagePrefsPtr) { 21 : distilled_page_prefs_(distillerPagePrefsPtr) {
19 } 22 }
20 23
21 DistilledPagePrefsAndroid::~DistilledPagePrefsAndroid() { 24 DistilledPagePrefsAndroid::~DistilledPagePrefsAndroid() {
(...skipping 12 matching lines...) Expand all
34 reinterpret_cast<DistilledPagePrefs*>(distilledPagePrefsPtr); 37 reinterpret_cast<DistilledPagePrefs*>(distilledPagePrefsPtr);
35 DistilledPagePrefsAndroid* distilled_page_prefs_android = 38 DistilledPagePrefsAndroid* distilled_page_prefs_android =
36 new DistilledPagePrefsAndroid(env, obj, distilledPagePrefs); 39 new DistilledPagePrefsAndroid(env, obj, distilledPagePrefs);
37 return reinterpret_cast<intptr_t>(distilled_page_prefs_android); 40 return reinterpret_cast<intptr_t>(distilled_page_prefs_android);
38 } 41 }
39 42
40 bool DistilledPagePrefsAndroid::Register(JNIEnv* env) { 43 bool DistilledPagePrefsAndroid::Register(JNIEnv* env) {
41 return RegisterNativesImpl(env); 44 return RegisterNativesImpl(env);
42 } 45 }
43 46
47 void DistilledPagePrefsAndroid::RemoveObserver(JNIEnv* env,
48 jobject obj,
49 jlong observerPtr) {
50 DistilledPagePrefsObserverWrapper* distilledPagePrefsObserverWrapper =
51 reinterpret_cast<DistilledPagePrefsObserverWrapper*>(observerPtr);
52 distilled_page_prefs_->RemoveObserver(distilledPagePrefsObserverWrapper);
53 }
54
55 void DistilledPagePrefsAndroid::AddObserver(JNIEnv* env,
56 jobject obj,
57 jlong observerPtr) {
58 DistilledPagePrefsObserverWrapper* distilledPagePrefsObserverWrapper =
59 reinterpret_cast<DistilledPagePrefsObserverWrapper*>(observerPtr);
60 distilled_page_prefs_->AddObserver(distilledPagePrefsObserverWrapper);
61 }
62
44 } // namespace android 63 } // namespace android
64
45 } // namespace dom_distiller 65 } // namespace dom_distiller
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698