[BackgroundSync] Trigger Background Sync events when Chrome is backgrounded on Android

content/public/android/java/src/org/chromium/content/browser/BackgroundSyncNetworkObserver.java

Index: content/public/android/java/src/org/chromium/content/browser/BackgroundSyncNetworkObserver.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/BackgroundSyncNetworkObserver.java b/content/public/android/java/src/org/chromium/content/browser/BackgroundSyncNetworkObserver.java
new file mode 100644
index 0000000000000000000000000000000000000000..2ee8c4969e681ec2c376eb00ee32728db023d97d
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/BackgroundSyncNetworkObserver.java
@@ -0,0 +1,50 @@
+// Copyright 2015 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.content.browser;
+
+import android.content.Context;
+
+import org.chromium.base.annotations.CalledByNative;
+import org.chromium.base.annotations.JNINamespace;
+import org.chromium.base.annotations.NativeClassQualifiedName;
+import org.chromium.net.NetworkChangeNotifierAutoDetect;
+
+/**
+ * Contains the Java code used by the BackgroundSyncNetworkObserverAndroid C++ class.
+ *
+ * Its purpose is to allow the BackgroundSyncManager to determine whether or not the device is
+ * actually online, for the purpose of triggering background sync events, without having to trigger

[jkarlin, 2015/08/28 01:19:48]

remove "actually". The last bit doesn't make much sense to me "without having to
trigger an online event while Chrome is backgrounded.

Perhaps:

The purpose of this class is to listen for and forward network connectivity
events to the BackgroundSyncNetworkObserver even when the application is paused.
The standard NetworkChangeNotifier does not listen for connectivity events when
the application is paused.

[iclelland, 2015/08/31 16:00:52]

Done.

+ * an online event while Chrome is backgrounded.
+ */
+@JNINamespace("content")
+public class BackgroundSyncNetworkObserver implements NetworkChangeNotifierAutoDetect.Observer {
+
+    private NetworkChangeNotifierAutoDetect mNotifier;
+    private long mNativePtr;
+
+    public BackgroundSyncNetworkObserver(Context ctx, long nativePtr) {
+        mNotifier = new NetworkChangeNotifierAutoDetect(this, ctx, true);
+        mNativePtr = nativePtr;
+        onConnectionTypeChanged(mNotifier.getCurrentConnectionType(
+                mNotifier.getCurrentNetworkState()));
+    }
+
+    @CalledByNative
+    public static BackgroundSyncNetworkObserver createObserver(Context ctx, long nativePtr) {
+        return new BackgroundSyncNetworkObserver(ctx, nativePtr);
+    }
+
+    @Override
+    public void onConnectionTypeChanged(int newConnectionType) {
+        nativeNotifyConnectionTypeChanged(mNativePtr, newConnectionType);
+    }
+
+    @Override
+    public void onMaxBandwidthChanged(double maxBandwidthMbps) {
+    }
+
+    @NativeClassQualifiedName("BackgroundSyncNetworkObserverAndroid")
+    private native void nativeNotifyConnectionTypeChanged(long nativePtr, int newConnectionType);
+}