Photo Picker dialog: Add UMA statistics.

chrome/android/java/src/org/chromium/chrome/browser/photo_picker/DecoderService.java

 // Copyright 2017 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.photo_picker;
 
 import android.app.Service;
 import android.content.Intent;
 import android.graphics.Bitmap;
 import android.os.Bundle;
 import android.os.Handler;
 import android.os.IBinder;
 import android.os.Message;
 import android.os.Messenger;
 import android.os.ParcelFileDescriptor;
 import android.os.RemoteException;
+import android.os.SystemClock;
 
 import org.chromium.base.Log;
 
 import java.io.FileDescriptor;
 import java.io.IOException;
 
 /**
  * A service to accept requests to take image file contents and decode them.
  */
 public class DecoderService extends Service {
     // Message ids for communicating with the client.
 
     // A message sent by the client to decode an image.
     static final int MSG_DECODE_IMAGE = 1;
     // A message sent by the server to notify the client of the results of the decoding.
     static final int MSG_IMAGE_DECODED_REPLY = 2;
 
     // The keys for the bundle when passing data to and from this service.
     static final String KEY_FILE_DESCRIPTOR = "file_descriptor";
     static final String KEY_FILE_PATH = "file_path";
     static final String KEY_IMAGE_BITMAP = "image_bitmap";
     static final String KEY_IMAGE_BYTE_COUNT = "image_byte_count";
     static final String KEY_IMAGE_DESCRIPTOR = "image_descriptor";
     static final String KEY_SIZE = "size";
     static final String KEY_SUCCESS = "success";
+    static final String KEY_DECODE_TIME = "decode_time";
 
     // A tag for logging error messages.
     private static final String TAG = "ImageDecoder";
 
     /**
      * Handler for incoming messages from clients.
      */
     static class IncomingHandler extends Handler {
         @Override
         public void handleMessage(Message msg) {
@@ skipping 11 matching lines @@
                         ParcelFileDescriptor pfd = payload.getParcelable(KEY_FILE_DESCRIPTOR);
                         size = payload.getInt(KEY_SIZE);
 
                         // Setup a minimum viable response to parent process. Will be fleshed out
                         // further below.
                         bundle = new Bundle();
                         bundle.putString(KEY_FILE_PATH, filePath);
                         bundle.putBoolean(KEY_SUCCESS, false);
 
                         FileDescriptor fd = pfd.getFileDescriptor();
+
+                        long begin = SystemClock.elapsedRealtime();
                         Bitmap bitmap = BitmapUtils.decodeBitmapFromFileDescriptor(fd, size);
+                        long decodeTime = SystemClock.elapsedRealtime() - begin;
+
                         try {
                             pfd.close();
                         } catch (IOException e) {
                             Log.e(TAG, "Closing failed " + filePath + " (size: " + size + ") " + e);
                         }
 
                         if (bitmap == null) {
                             Log.e(TAG, "Decode failed " + filePath + " (size: " + size + ")");
                             sendReply(client, bundle); // Sends SUCCESS == false;
                             return;
                         }
 
                         // The most widely supported, easiest, and reasonably efficient method is to
                         // decode to an immutable bitmap and just return the bitmap over binder. It
                         // will internally memcpy itself to ashmem and then just send over the file
                         // descriptor. In the receiving process it will just leave the bitmap on
                         // ashmem since it's immutable and carry on.
                         bundle.putParcelable(KEY_IMAGE_BITMAP, bitmap);
                         bundle.putBoolean(KEY_SUCCESS, true);
+                        bundle.putLong(KEY_DECODE_TIME, decodeTime);
                         sendReply(client, bundle);
                         bitmap.recycle();
                     } catch (Exception e) {
                         // This service has no UI and maintains no state so if it crashes on
                         // decoding a photo, it is better UX to eat the exception instead of showing
                         // a crash dialog and discarding other requests that have already been sent.
                         Log.e(TAG,
                                 "Unexpected error during decoding " + filePath + " (size: " + size
                                         + ") " + e);
 
@@ skipping 23 matching lines @@
 
     /**
      * When binding to the service, we return an interface to our messenger
      * for sending messages to the service.
      */
     @Override
     public IBinder onBind(Intent intent) {
         return mMessenger.getBinder();
     }
 }