Connecting UserClassifier to NtpSnippetsFetcher

components/ntp_snippets/user_classifier.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_USER_CLASSIFIER_H_
 #define COMPONENTS_NTP_SNIPPETS_USER_CLASSIFIER_H_
 
 #include <string>
 
 #include "base/macros.h"
@@ skipping 48 matching lines @@
   // Informs the UserClassifier about a new event for |metric|. The
   // classification is based on these calls.
   void OnEvent(Metric metric);
 
   // Get the estimate average length of the interval between two successive
   // events of the given type.
   double GetEstimatedAvgTime(Metric metric) const;
 
   // Return the classification of the current user.
   UserClass GetUserClass() const;
+  std::string GetUserClassNameForProto() const;

[Marc Treib, 2016/10/06 13:41:55]

What does this have to do with protobufs?
Should this live in the fetcher instead? The rest of the server protocol is
defined there.

[jkrcal, 2016/10/06 14:45:58]

Makes sense. Done.

   std::string GetUserClassDescriptionForDebugging() const;
 
   // Resets the classification (emulates a fresh upgrade / install).
   void ClearClassificationForDebugging();
 
  private:
   // The event has happened, recompute the metric accordingly. Then store and
   // return the new value.
   double UpdateMetricOnEvent(Metric metric);
   // No event has happened but we need to get up-to-date metric, recompute and
@@ skipping 21 matching lines @@
   // Params of the classification.
   const double active_consumer_scrolls_at_least_once_per_hours_;
   const double rare_user_opens_ntp_at_most_once_per_hours_;
 
   DISALLOW_COPY_AND_ASSIGN(UserClassifier);
 };
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_USER_CLASSIFIER_H_