summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndroid (Google) Code Review <android-gerrit@google.com>2009-05-20 15:28:36 -0700
committerAndroid (Google) Code Review <android-gerrit@google.com>2009-05-20 15:28:36 -0700
commitaa9833885d89d42a0cd0b4202485e67b63d51412 (patch)
treea6087980f6486eafadd76f5cda152d4dd3049465
parent009b32dc8b9c32160e02a63b60636854f0ceb86f (diff)
parent3c1664216d3754674534932716b41b69999d8013 (diff)
downloadframeworks_base-aa9833885d89d42a0cd0b4202485e67b63d51412.zip
frameworks_base-aa9833885d89d42a0cd0b4202485e67b63d51412.tar.gz
frameworks_base-aa9833885d89d42a0cd0b4202485e67b63d51412.tar.bz2
Merge change 1932 into donut
* changes: Qualify a type and add some explanatory comments.
-rw-r--r--core/java/com/google/android/net/GoogleHttpClient.java6
1 files changed, 5 insertions, 1 deletions
diff --git a/core/java/com/google/android/net/GoogleHttpClient.java b/core/java/com/google/android/net/GoogleHttpClient.java
index 25d0122..922f5be 100644
--- a/core/java/com/google/android/net/GoogleHttpClient.java
+++ b/core/java/com/google/android/net/GoogleHttpClient.java
@@ -73,7 +73,7 @@ public class GoogleHttpClient implements HttpClient {
private final AndroidHttpClient mClient;
private final ContentResolver mResolver;
private final String mAppName, mUserAgent;
- private final ThreadLocal mConnectionAllocated = new ThreadLocal<Boolean>();
+ private final ThreadLocal<Boolean> mConnectionAllocated = new ThreadLocal<Boolean>();
/**
* Create an HTTP client without SSL session persistence.
@@ -153,6 +153,8 @@ public class GoogleHttpClient implements HttpClient {
* Delegating wrapper for SocketFactory records when sockets are connected.
* We use this to know whether a connection was created vs reused, to
* gather per-app statistics about connection reuse rates.
+ * (Note, we record only *connection*, not *creation* of sockets --
+ * what we care about is the network overhead of an actual TCP connect.)
*/
private class WrappedSocketFactory implements SocketFactory {
private SocketFactory mDelegate;
@@ -227,6 +229,8 @@ public class GoogleHttpClient implements HttpClient {
} finally {
// Record some statistics to the checkin service about the outcome.
// Note that this is only describing execute(), not body download.
+ // We assume the database writes are much faster than network I/O,
+ // and not worth running in a background thread or anything.
try {
long elapsed = SystemClock.elapsedRealtime() - start;
ContentValues values = new ContentValues();