Open source Tabs.* UMA stats

Open source Tabs.* UMA stats

BUG=223859
R=jamescook@chromium.org, isherman@chromium.org

Committed: https://crrev.com/57c64f59502ede094c8082401513894027c0a563
Cr-Commit-Position: refs/heads/master@{#332557}

[Ilya Sherman, 2015-06-02 22:28:59]

Thanks!  Happy to stamp this once you also send the inverse CL in google3.

[oshima, 2015-06-02 22:59:49]

I added obsolete ones (SadTab.Displayed, SadTab.Created). PTAL

[Ilya Sherman, 2015-06-02 23:00:28]

LGTM, thanks.

[oshima, 2015-06-02 23:02:32]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2015-06-02 23:07:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1143253011/20001

[commit-bot: I haz the power, 2015-06-02 23:44:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-02 23:44:22]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chromium-reviews, 2015-06-02 23:45:27]

Can you hold off on committing this? I'm having trouble on the google3 side.


On Tue, Jun 2, 2015 at 4:44 PM, 'commit-bot: I haz the power' via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com> wrote:

> Try jobs failed on following builders:
>   linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
>
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...
> )
>
> https://codereview.chromium.org/1143253011/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[oshima, 2015-06-02 23:52:59]

On 2015/06/02 23:45:27, chromium-reviews wrote:
> Can you hold off on committing this? I'm having trouble on the google3 side.

FYI: CQ stopped for different reasons.

> 
> 
> On Tue, Jun 2, 2015 at 4:44 PM, 'commit-bot: I haz the power' via
> http://codereview.chromium.org
<mailto:reply@chromiumcodereview-hr.appspotmail.com> wrote:
> 
> > Try jobs failed on following builders:
> >   linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
> >
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...
> > )
> >
> > https://codereview.chromium.org/1143253011/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[oshima, 2015-06-03 00:58:28]

uploaded new patch that incorporates new changes in
https://critique.corp.google.com/#review/95061491

PTAL

[Ilya Sherman, 2015-06-03 03:13:55]

LGTM

[oshima, 2015-06-03 03:51:46]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2015-06-03 03:53:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1143253011/40001

[commit-bot: I haz the power, 2015-06-03 05:16:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-06-03 05:17:30]

Patchset 3 (id:??) landed as
https://crrev.com/57c64f59502ede094c8082401513894027c0a563
Cr-Commit-Position: refs/heads/master@{#332557}