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

Unified Diff: net/android/network_change_notifier_android.h

Issue 10928193: Add native-side unit test for Android NetworkChangeNotifier (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Removed Java-side observers for now Created 8 years, 3 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: net/android/network_change_notifier_android.h
diff --git a/net/android/network_change_notifier_android.h b/net/android/network_change_notifier_android.h
index 3c71dc5995bb31fdfacf88b46fc8e37ce2cb8106..79a2291ef7351973c9f339c32439b11ef08b9580 100644
--- a/net/android/network_change_notifier_android.h
+++ b/net/android/network_change_notifier_android.h
@@ -18,13 +18,13 @@ class NetworkChangeNotifier : public net::NetworkChangeNotifier {
NetworkChangeNotifier();
virtual ~NetworkChangeNotifier();
- void NotifyObservers(JNIEnv* env, jobject obj);
+ void NotifyObserversOfConnectionTypeChange(JNIEnv* env, jobject obj);
+
+ void ForceConnectivityState(bool state);
szym 2012/09/14 19:07:29 What is this for? If this is for tests only, at le
gone 2012/09/15 01:01:55 Done.
static bool Register(JNIEnv* env);
private:
- void CreateJavaObject(JNIEnv* env);
-
// NetworkChangeNotifier:
szym 2012/09/14 19:07:29 since you're already here, make it "net::NetworkCh
gone 2012/09/15 01:01:55 Appended the Android suffix.
virtual net::NetworkChangeNotifier::ConnectionType
GetCurrentConnectionType() const OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698