Hi Charles, here is one of the Site Engagement metrics bugs I mentioned. Dom or I can help you get started if you have any questions.

Hi Kendra, what is this bucket 0-1? Can you point me to the code where SES=1 is counted?

I believe you can find that engagement metrics here: https://cs.chromium.org/chromium/src/chrome/browser/engagement/site_engagement_metrics.cc

I don't think we count SES=1 explicitly, we just have the histogram.

Thanks,
Kendra

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/169ed39de473bb196dcb5960e4c19ced3336de4e

commit 169ed39de473bb196dcb5960e4c19ced3336de4e
Author: charleszhao <charleszhao@chromium.org>
Date: Mon Jan 30 03:50:34 2017

(1) Add a new histogram for EngagementScore to check whether it's zero
or non-zero.

(2) The score is treated as 0 if it's less than  ScoreCleanupThreshold.

BUG= 656834 

Review-Url: https://codereview.chromium.org/2633283002
Cr-Commit-Position: refs/heads/master@{#446956}

[modify] https://crrev.com/169ed39de473bb196dcb5960e4c19ced3336de4e/chrome/browser/engagement/site_engagement_metrics.cc
[modify] https://crrev.com/169ed39de473bb196dcb5960e4c19ced3336de4e/chrome/browser/engagement/site_engagement_metrics.h
[modify] https://crrev.com/169ed39de473bb196dcb5960e4c19ced3336de4e/tools/metrics/histograms/histograms.xml