[Custom Tabs] Postpone RecordAction until native libray is loaded

chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchMetrics.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.metrics;
 
 import android.util.Pair;
 
 import org.chromium.base.annotations.JNINamespace;
 import org.chromium.base.metrics.RecordHistogram;
+import org.chromium.base.metrics.RecordUserAction;
 import org.chromium.content_public.browser.WebContents;
 
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
 /**
  * Used for recording metrics about Chrome launches that need to be recorded before the native
  * library may have been loaded.  Metrics are cached until the library is known to be loaded, then
  * committed to the MetricsService all at once.
@@ skipping 15 matching lines @@
          */
         protected CachedHistogram(String histogramName) {
             mHistogramName = histogramName;
             sEvents.add(this);
         }
 
         /** Commits the histogram. Expects the native library to be loaded. */
         protected abstract void commitAndClear();
     }
 
+    /**
+     * Caches an action that will be recorded after native side is loaded.
+     */
+    public static class ActionEvent extends CachedHistogram {
+

[gone, 2016/04/27 00:30:09]

private boolean mNeedsToBeRecorded = true;


protected void commitAndClear() {
    if (mNeedsToBeRecorded) RecordUserAction.record(mHistogramName);
    mNeedsToBeRecorded = false;
}

[Ian Wen, 2016/04/27 00:39:44]

Done.

+        public ActionEvent(String actionName) {
+            super(actionName);
+        }
+
+        @Override
+        protected void commitAndClear() {
+            RecordUserAction.record(mHistogramName);
+        }
+    }
+
     /** Caches whether an event happened. */
     public static class BooleanEvent extends CachedHistogram {
         private boolean mIsHit;
 
         public BooleanEvent(String histogramName) {
             super(histogramName);
         }
 
         /** Records that the histogram condition occurred. */
         public void recordHit() {
@@ skipping 111 matching lines @@
         }
         sWebappHistogramTimes.clear();
 
         // Record generic cached events.
         for (CachedHistogram event : CachedHistogram.sEvents) event.commitAndClear();
     }
 
     private static native void nativeRecordLaunch(
             boolean standalone, String url, int source, WebContents webContents);
 }