Index: chrome/browser/notifications/notification_platform_bridge_android.cc |
diff --git a/chrome/browser/notifications/notification_platform_bridge_android.cc b/chrome/browser/notifications/notification_platform_bridge_android.cc |
index 3f64247f468c31551ca6ff2e75e03b76d7b7c63f..0c8c82fe6f819c043ad8bc8419020a02332cd819 100644 |
--- a/chrome/browser/notifications/notification_platform_bridge_android.cc |
+++ b/chrome/browser/notifications/notification_platform_bridge_android.cc |
@@ -7,7 +7,6 @@ |
#include <utility> |
#include <vector> |
-#include "base/android/build_info.h" |
#include "base/android/jni_array.h" |
#include "base/android/jni_string.h" |
#include "base/command_line.h" |
@@ -316,30 +315,8 @@ |
const std::string& profile_id, |
bool incognito, |
std::set<std::string>* notifications) const { |
- DCHECK(notifications); |
- JNIEnv* env = AttachCurrentThread(); |
- |
- // Android only supports retrieving existing notifications from M+ |
- if (base::android::BuildInfo::GetInstance()->sdk_int() < |
- base::android::SDK_VERSION_MARSHMALLOW) { |
- return false; |
- } |
- |
- const ScopedJavaLocalRef<jstring> j_profile_id = |
- ConvertUTF8ToJavaString(env, profile_id); |
- |
- ScopedJavaLocalRef<jobjectArray> j_notification_ids = |
- Java_NotificationPlatformBridge_getNotificationsForProfile( |
- env, java_object_, j_profile_id); |
- if (j_notification_ids.obj()) { |
- std::vector<std::string> notification_ids; |
- base::android::AppendJavaStringArrayToStringVector( |
- env, j_notification_ids.obj(), ¬ification_ids); |
- for (const auto& id : notification_ids) { |
- notifications->insert(id); |
- } |
- } |
- return true; |
+ // TODO(miguelg): This can actually be implemented for M+ |
+ return false; |
} |
// static |