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

Unified Diff: libadblockplus-android/jni/JniShowNotificationCallback.cpp

Issue 29422558: Issue 5167 - Update to use libadblockplus revision dca8df9af1a7 (Closed)
Patch Set: updated dependencies (..-binaries) Created May 10, 2017, 6:54 a.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
« no previous file with comments | « libadblockplus-android/jni/JniNotification.cpp ('k') | libadblockplus-android/jni/JniSubscription.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: libadblockplus-android/jni/JniShowNotificationCallback.cpp
diff --git a/libadblockplus-android/jni/JniShowNotificationCallback.cpp b/libadblockplus-android/jni/JniShowNotificationCallback.cpp
index 0bd980a5e9d3d3bab924aedcba989e9d67c8b3cb..605bb939551ac69f67d88941a2b4e11debb32515 100644
--- a/libadblockplus-android/jni/JniShowNotificationCallback.cpp
+++ b/libadblockplus-android/jni/JniShowNotificationCallback.cpp
@@ -37,8 +37,7 @@ JniShowNotificationCallback::JniShowNotificationCallback(JNIEnv* env,
{
}
-void JniShowNotificationCallback::Callback(
- const AdblockPlus::NotificationPtr& notificationPtr)
+void JniShowNotificationCallback::Callback(AdblockPlus::Notification&& notification)
{
JNIEnvAcquire env(GetJavaVM());
@@ -49,8 +48,7 @@ void JniShowNotificationCallback::Callback(
if (method)
{
- JniLocalReference<jobject> jNotification(*env, NewJniNotification(*env,
- notificationPtr));
+ JniLocalReference<jobject> jNotification(*env, NewJniNotification(*env, std::move(notification)));
env->CallVoidMethod(GetCallbackObject(), method, *jNotification);
}
« no previous file with comments | « libadblockplus-android/jni/JniNotification.cpp ('k') | libadblockplus-android/jni/JniSubscription.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld