Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java |
| index 81feaa909ac1511b3d4dd9f2c1620bd1adea379a..2fe80c1f031f55424db3416df14489aa3c99410c 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java |
| @@ -13,6 +13,7 @@ import android.content.pm.PackageManager; |
| import android.content.pm.PackageManager.NameNotFoundException; |
| import android.os.Binder; |
| import android.os.StrictMode; |
| +import android.os.SystemClock; |
| import android.text.TextUtils; |
| import com.google.android.gms.common.ConnectionResult; |
| @@ -25,6 +26,7 @@ import org.chromium.base.VisibleForTesting; |
| import org.chromium.base.metrics.RecordHistogram; |
| import org.chromium.chrome.browser.ChromeApplication; |
| +import java.util.concurrent.TimeUnit; |
| import java.util.concurrent.atomic.AtomicReference; |
| /** |
| @@ -243,7 +245,12 @@ public class ExternalAuthUtils { |
| StrictMode.ThreadPolicy oldPolicy = StrictMode.allowThreadDiskReads(); |
| StrictMode.allowThreadDiskWrites(); |
| try { |
| - return GoogleApiAvailability.getInstance().isGooglePlayServicesAvailable(context); |
| + long time = SystemClock.elapsedRealtime(); |
| + int isAvailable = |
| + GoogleApiAvailability.getInstance().isGooglePlayServicesAvailable(context); |
|
Yaron
2016/01/21 22:03:22
this one also can happen without native
Peter Wen
2016/01/21 22:13:54
Done.
|
| + RecordHistogram.recordTimesHistogram("Android.StrictMode.CheckGooglePlayServicesTime", |
| + SystemClock.elapsedRealtime() - time, TimeUnit.MILLISECONDS); |
| + return isAvailable; |
| } finally { |
| StrictMode.setThreadPolicy(oldPolicy); |
| } |