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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/services/gcm/GCMBackgroundTask.java

Issue 2722503004: Route through the BGTaskScheduler when receiving a message for the GCM Driver (Closed)
Patch Set: Route through the BGTaskScheduler when receiving a message for the GCM Driver Created 3 years, 10 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/src/org/chromium/chrome/browser/services/gcm/GCMBackgroundTask.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/services/gcm/GCMBackgroundTask.java b/chrome/android/java/src/org/chromium/chrome/browser/services/gcm/GCMBackgroundTask.java
new file mode 100644
index 0000000000000000000000000000000000000000..db2533bc9c1656448b2cb88bfc66875c37bbdbff
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/services/gcm/GCMBackgroundTask.java
@@ -0,0 +1,58 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.chrome.browser.services.gcm;
+
+import android.annotation.TargetApi;
+import android.content.Context;
+import android.os.Build;
+import android.os.Bundle;
+import android.support.annotation.MainThread;
+
+import org.chromium.base.Log;
+import org.chromium.components.background_task_scheduler.BackgroundTask;
+import org.chromium.components.background_task_scheduler.TaskParameters;
+import org.chromium.components.gcm_driver.GCMMessage;
+
+/**
+ * Processes jobs that have been scheduled for delivering GCM messages to the native GCM Driver,
+ * processing for which may exceed the lifetime of the GcmListenerService.
+ */
+@TargetApi(Build.VERSION_CODES.N)
+public class GCMBackgroundTask implements BackgroundTask {
+ private static final String TAG = "GCMBackgroundTask";
+
+ /**
+ * Called when a GCM message is ready to be delivered to the GCM Driver consumer. Because we
+ * don't yet know when a message has been fully processed, the task returns that processing has
+ * been completed, and we hope that the system keeps us alive long enough to finish processing.
+ *
+ * @return Boolean indicating whether the WakeLock for this task must be maintained.
+ */
+ @MainThread
+ @Override
+ public boolean onStartTask(
+ Context context, TaskParameters taskParameters, TaskFinishedCallback callback) {
+ Bundle extras = taskParameters.getExtras();
+ if (!GCMMessage.validateBundle(extras)) {
+ Log.e(TAG, "The received bundle containing message data could not be validated.");
+ return false;
+ }
+
+ ChromeGcmListenerService.dispatchMessageToDriver(context, new GCMMessage(extras));
+ return false;
+ }
+
+ /**
+ * Called when the system has determined that processing the GCM message must be stopped.
+ *
+ * @return Boolean indicating whether the task has to be rescheduled.
+ */
+ @MainThread
+ @Override
+ public boolean onStopTask(Context context, TaskParameters taskParameters) {
+ // The GCM Driver has no mechanism for aborting previously dispatched messages.
+ return false;
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698