Add GetTotalEngagementPoints to SiteEngagementService.

chrome/browser/engagement/site_engagement_service_unittest.cc

 // 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.
 
 #include "base/command_line.h"
 #include "base/test/simple_test_clock.h"
 #include "base/values.h"
 #include "chrome/browser/engagement/site_engagement_helper.h"
 #include "chrome/browser/engagement/site_engagement_service.h"
 #include "chrome/browser/engagement/site_engagement_service_factory.h"
@@ skipping 308 matching lines @@
 
   SiteEngagementService* service =
       SiteEngagementServiceFactory::GetForProfile(profile());
   DCHECK(service);
 
   GURL url("http://www.google.com/");
 
   AddTab(browser(), GURL("about:blank"));
   EXPECT_EQ(0, service->GetScore(url));
   int prev_score = service->GetScore(url);
+  int prev_total_score = service->GetTotalEngagementPoints();
 
   NavigateAndCommitActiveTab(url);
   EXPECT_LT(prev_score, service->GetScore(url));
   prev_score = service->GetScore(url);
+  EXPECT_LT(prev_total_score, service->GetTotalEngagementPoints());
+  prev_total_score = service->GetTotalEngagementPoints();
 
   NavigateAndCommitActiveTab(url);
   EXPECT_LT(prev_score, service->GetScore(url));
+  EXPECT_LT(prev_total_score, service->GetTotalEngagementPoints());
 }

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

Could we add a new test where we browse to a bunch of sites and check that the
individual scores (service->GetScore) add up to the total score?

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

Yep, I just needed to check with benwells why this didn't use real numbers and
stuff.