Make sure that only instant apps API time is recorded and not anything else. Currently there's a path where if an exception is thrown, we could be recording an incorrect value.
The following revision refers to this bug: https://chrome-internal.googlesource.com/clank/internal/apps/+/c9f760a8c4620ed7b13031ccd233f0fc95cbe6e2 commit c9f760a8c4620ed7b13031ccd233f0fc95cbe6e2 Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Mon Dec 05 22:29:11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ba6c108d584629567b1bf45515b20ba90d6dee8d commit ba6c108d584629567b1bf45515b20ba90d6dee8d Author: mariakhomenko <mariakhomenko@chromium.org> Date: Tue Dec 06 20:48:35 2016 Instant Apps: Change the name of the metric. The metric is now recorded differently, update to a new name. BUG= 670898 Review-Url: https://codereview.chromium.org/2551073003 Cr-Commit-Position: refs/heads/master@{#436711} [modify] https://crrev.com/ba6c108d584629567b1bf45515b20ba90d6dee8d/chrome/android/java/src/org/chromium/chrome/browser/instantapps/InstantAppsHandler.java [modify] https://crrev.com/ba6c108d584629567b1bf45515b20ba90d6dee8d/tools/metrics/histograms/histograms.xml
Your change meets the bar and is auto-approved for M56 (branch: 2924)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9595405d19bf5e40d885f753afe7ed47f6081b52 commit 9595405d19bf5e40d885f753afe7ed47f6081b52 Author: mariakhomenko <mariakhomenko@chromium.org> Date: Wed Dec 07 19:41:29 2016 Instant Apps: Split latency recordings into two. Separately record the latency for Instant Apps API calls that result in an application being found and those that do not. BUG= 670898 Review-Url: https://codereview.chromium.org/2555693003 Cr-Commit-Position: refs/heads/master@{#437034} [modify] https://crrev.com/9595405d19bf5e40d885f753afe7ed47f6081b52/chrome/android/java/src/org/chromium/chrome/browser/instantapps/InstantAppsHandler.java [modify] https://crrev.com/9595405d19bf5e40d885f753afe7ed47f6081b52/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chrome-internal.googlesource.com/clank/internal/apps/+/ca7a6ef99c6fee55a091d3173e921326e01bfa1a commit ca7a6ef99c6fee55a091d3173e921326e01bfa1a Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Wed Dec 07 00:18:25 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/clank/internal/apps/+/322d736026e79562d8df3df210283b2f12503ce2 commit 322d736026e79562d8df3df210283b2f12503ce2 Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Mon Dec 05 22:29:11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2aeb160a4b36d36d2f771a616bf8da9da21f4cc8 commit 2aeb160a4b36d36d2f771a616bf8da9da21f4cc8 Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Thu Dec 08 18:31:29 2016 Instant Apps: Change the name of the metric. The metric is now recorded differently, update to a new name. BUG= 670898 Review-Url: https://codereview.chromium.org/2551073003 Cr-Commit-Position: refs/heads/master@{#436711} (cherry picked from commit ba6c108d584629567b1bf45515b20ba90d6dee8d) Review URL: https://codereview.chromium.org/2550413006 . Cr-Commit-Position: refs/branch-heads/2924@{#412} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/2aeb160a4b36d36d2f771a616bf8da9da21f4cc8/chrome/android/java/src/org/chromium/chrome/browser/instantapps/InstantAppsHandler.java [modify] https://crrev.com/2aeb160a4b36d36d2f771a616bf8da9da21f4cc8/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ca33327934e6b9fdff331bfa2b1768b1420616b2 commit ca33327934e6b9fdff331bfa2b1768b1420616b2 Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Thu Dec 08 19:37:56 2016 Instant Apps: Split latency recordings into two. Separately record the latency for Instant Apps API calls that result in an application being found and those that do not. BUG= 670898 Review-Url: https://codereview.chromium.org/2555693003 Cr-Commit-Position: refs/heads/master@{#437034} (cherry picked from commit 9595405d19bf5e40d885f753afe7ed47f6081b52) Review URL: https://codereview.chromium.org/2561863002 . Cr-Commit-Position: refs/branch-heads/2924@{#414} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/ca33327934e6b9fdff331bfa2b1768b1420616b2/chrome/android/java/src/org/chromium/chrome/browser/instantapps/InstantAppsHandler.java [modify] https://crrev.com/ca33327934e6b9fdff331bfa2b1768b1420616b2/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chrome-internal.googlesource.com/clank/internal/apps/+/dad4ad1a699355716512456a8170f300b22f01e9 commit dad4ad1a699355716512456a8170f300b22f01e9 Author: Maria Khomenko <mariakhomenko@chromium.org> Date: Wed Dec 07 00:18:25 2016
Comment 1 by bugdroid1@chromium.org
, Dec 5 2016