Add new app banner metrics using InstallableStatusCode.

chrome/browser/banners/app_banner_settings_helper.h

 // Copyright 2014 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 CHROME_BROWSER_BANNERS_APP_BANNER_SETTINGS_HELPER_H_
 #define CHROME_BROWSER_BANNERS_APP_BANNER_SETTINGS_HELPER_H_
 
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/time/time.h"
+#include "chrome/browser/installable/installable_logging.h"
 #include "ui/base/page_transition_types.h"
 
 namespace content {
 class WebContents;
 }  // namespace content
 
 class GURL;
 class Profile;
 
 // Utility class to record banner events for the given package or start url.
@@ skipping 64 matching lines @@
 
   // Record a banner could show event, with a specified transition type.
   static void RecordBannerCouldShowEvent(
       content::WebContents* web_contents,
       const GURL& origin_url,
       const std::string& package_name_or_start_url,
       base::Time time,
       ui::PageTransition transition_type);
 
   // Determine if the banner should be shown, given the recorded events for the
-  // supplied app.
-  static bool ShouldShowBanner(content::WebContents* web_contents,
-                               const GURL& origin_url,
-                               const std::string& package_name_or_start_url,
-                               base::Time time);
+  // supplied app. Returns an InstallableStatusCode indicated the reason why the
+  // banner shouldn't be shown, or NO_ERROR_DETECTED if it should be shown.
+  static InstallableStatusCode ShouldShowBanner(
+      content::WebContents* web_contents,
+      const GURL& origin_url,
+      const std::string& package_name_or_start_url,
+      base::Time time);
 
   // Gets the could have been shown events that are stored for the given package
   // or start url. This is only exposed for testing.
   static std::vector<BannerEvent> GetCouldShowBannerEvents(
       content::WebContents* web_contents,
       const GURL& origin_url,
       const std::string& package_name_or_start_url);
 
   // Get the recorded event for an event type that only records the last event.
   // Should not be used with APP_BANNER_EVENT_COULD_SHOW. This is only exposed
@@ skipping 51 matching lines @@
 
   // Returns true if the app banner trigger condition should use the site
   // engagement score instead of the navigation-based heuristic.
   static bool ShouldUseSiteEngagementScore();
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(AppBannerSettingsHelper);
 };
 
 #endif  // CHROME_BROWSER_BANNERS_APP_BANNER_SETTINGS_HELPER_H_