Add GetTotalEngagementPoints to SiteEngagementService.

chrome/browser/engagement/site_engagement_service.cc

Index: chrome/browser/engagement/site_engagement_service.cc
diff --git a/chrome/browser/engagement/site_engagement_service.cc b/chrome/browser/engagement/site_engagement_service.cc
index 2885fbfc99fce5ca781d3de6d15448270fbb09da..0071552ef52e08012f894e330570cfb5e93e0554 100644
--- a/chrome/browser/engagement/site_engagement_service.cc
+++ b/chrome/browser/engagement/site_engagement_service.cc
@@ -192,3 +192,17 @@ int SiteEngagementService::GetScore(const GURL& url) {
   return score.Score();
 }
 
+int SiteEngagementService::GetTotalEngagementPoints() {
+  HostContentSettingsMap* settings_map = profile_->GetHostContentSettingsMap();
+  ContentSettingsForOneType engagement_settings;
+  settings_map->GetSettingsForOneType(CONTENT_SETTINGS_TYPE_SITE_ENGAGEMENT,
+                                      std::string(), &engagement_settings);
+  int total_score = 0;
+  for (auto it : engagement_settings) {

[raymes, 2015/07/01 05:33:08]

perhaps 
for (auto site : engagement_settings)

Also I think you can use "const auto& site" here to just grab a reference.

[calamity, 2015/07/06 01:51:06]

Done.

+    scoped_ptr<base::DictionaryValue> settings =
+        GetOriginDict(settings_map, GURL(it.primary_pattern.ToString()));

[raymes, 2015/07/01 05:33:09]

I guess we should technically check that the GURL is valid first (since
ContentSettingsPattern isn't necessarily a url (it may be a pattern). In our
case it /should/ be a URL but there isn't anything that enforces that (and it's
being read from a pref on disk) so we should check.

[calamity, 2015/07/06 01:51:06]

Done.

+    SiteEngagementScore score(&clock_, *settings);
+    total_score += score.Score();
+  }
+  return total_score;
+}