Measure time spent during StrictMode violations.

chrome/android/java/src/org/chromium/chrome/browser/externalauth/ExternalAuthUtils.java

 // 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.chrome.browser.externalauth;
 
 import android.app.Activity;
 import android.app.Dialog;
 import android.content.Context;
 import android.content.DialogInterface.OnCancelListener;
 import android.content.pm.ApplicationInfo;
 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;
 import com.google.android.gms.common.GoogleApiAvailability;
 
 import org.chromium.base.ApplicationStatus;
 import org.chromium.base.Log;
 import org.chromium.base.ThreadUtils;
 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;
 
 /**
  * Utility class for external authentication tools.
  *
  * This class is safe to use on any thread.
  */
 public class ExternalAuthUtils {
     public static final int FLAG_SHOULD_BE_GOOGLE_SIGNED = 1 << 0;
     public static final int FLAG_SHOULD_BE_SYSTEM = 1 << 1;
@@ skipping 198 matching lines @@
      * and returns the code produced by the attempt. Subclasses can override to force the behavior
      * one way or another, or to change the way that the check is performed.
      * @param context The current context.
      * @return The code produced by calling the external code
      */
     protected int checkGooglePlayServicesAvailable(final Context context) {
         // Temporarily allowing disk access. TODO: Fix. See http://crbug.com/577190
         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);
         }
     }
 
     /**
      * Invokes whatever external code is necessary to check if the specified result code from
      * {@link #checkGooglePlayServicesAvailable(Context)} represents success.
      * @param resultCode The code to check
      * @return true If the result code represents success.
@@ skipping 46 matching lines @@
      */
     public void showErrorDialog(final int errorCode, final Activity activity, final int requestCode,
             final OnCancelListener onCancelListener) {
         final Dialog dialog = GoogleApiAvailability.getInstance().getErrorDialog(
                 activity, errorCode, requestCode, onCancelListener);
         if (dialog != null) { // This can happen if errorCode is ConnectionResult.SERVICE_INVALID
             dialog.show();
         }
     }
 }