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

Unified Diff: chrome/android/java/AndroidManifest.xml

Issue 2611333002: Android web notifications: Schedule job instead of starting service (N+) (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/AndroidManifest.xml
diff --git a/chrome/android/java/AndroidManifest.xml b/chrome/android/java/AndroidManifest.xml
index 02ecd0d2d99efb7c9ffa9657d0dee2ddf465e47d..10ef9a2f4870d69f88ca179ff30b60db0c72495f 100644
--- a/chrome/android/java/AndroidManifest.xml
+++ b/chrome/android/java/AndroidManifest.xml
@@ -786,7 +786,8 @@ by a child template that "extends" this file.
<!-- Android Notification service listener -->
<service android:name="org.chromium.chrome.browser.notifications.NotificationService"
- android:exported="false"/>
+ android:exported="false"
+ android:permission="android.permission.BIND_JOB_SERVICE"/>
Peter Beverloo 2017/01/09 14:28:05 This is API level 21, will it be ignored by earlie
awdf 2017/01/11 14:56:17 I think this would just be ignored. Still installs
<receiver android:name="org.chromium.chrome.browser.notifications.NotificationService$Receiver"
android:exported="false">
<intent-filter>

Powered by Google App Engine
This is Rietveld 408576698