Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: jni/JniCallbacks.h

Issue 5697499218051072: Usage of new API, cleanups (reduced) (Closed)
Patch Set: Created April 11, 2014, 1:31 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: jni/JniCallbacks.h
diff --git a/jni/JniCallbacks.h b/jni/JniCallbacks.h
index 38e520bc53732fe3ca0bb68a093426be5f0556f6..afa747615c57dd0da1cf5b1f2d05ac6dfaf1227b 100644
--- a/jni/JniCallbacks.h
+++ b/jni/JniCallbacks.h
@@ -65,6 +65,9 @@ class JniFilterChangeCallback : public JniCallbackBase
public:
JniFilterChangeCallback(JNIEnv* env, jobject callbackObject);
void Callback(const std::string& arg, const AdblockPlus::JsValuePtr jsValue);
+
+private:
+ JniGlobalReference<jclass>::Ptr jsValueClass;
Felix Dahlke 2014/04/16 15:24:25 You seem to make everything const that can be, so
René Jeschke 2014/04/16 17:51:47 Done.
};
class JniLogSystemCallback : public JniCallbackBase, public AdblockPlus::LogSystem

Powered by Google App Engine
This is Rietveld