Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(507)

Side by Side Diff: chrome/browser/interstitials/chrome_metrics_helper.h

Issue 2575623002: Componentizing SafeBrowsingBlockingPage Part 1 (Closed)
Patch Set: address comments from nparker@ Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_ 5 #ifndef CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_
6 #define CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_ 6 #define CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "chrome/common/features.h" 11 #include "chrome/common/features.h"
12 #include "components/security_interstitials/core/metrics_helper.h" 12 #include "components/security_interstitials/core/metrics_helper.h"
13 #include "extensions/features/features.h" 13 #include "extensions/features/features.h"
14 #include "url/gurl.h" 14 #include "url/gurl.h"
15 15
16 namespace content { 16 namespace content {
17 class WebContents; 17 class WebContents;
18 } 18 }
19 19
20 namespace extensions { 20 namespace extensions {
21 class ExperienceSamplingEvent; 21 class ExperienceSamplingEvent;
22 } 22 }
23 23
24 class CaptivePortalMetricsRecorder; 24 class CaptivePortalMetricsRecorder;
25 25
26 // This class adds desktop-Chrome-specific metrics (extension experience 26 // This class adds desktop-Chrome-specific metrics (extension experience
27 // sampling) to the security_interstitials::MetricsHelper. Together, they 27 // sampling) to the security_interstitials::MetricsHelper. Together, they
28 // record UMA, Rappor, and experience sampling metrics. 28 // record UMA, and experience sampling metrics.
29 29
30 // This class is meant to be used on the UI thread for captive portal metrics. 30 // This class is meant to be used on the UI thread for captive portal metrics.
31 class ChromeMetricsHelper : public security_interstitials::MetricsHelper { 31 class ChromeMetricsHelper : public security_interstitials::MetricsHelper {
32 public: 32 public:
33 ChromeMetricsHelper( 33 ChromeMetricsHelper(
34 content::WebContents* web_contents, 34 content::WebContents* web_contents,
35 const GURL& url, 35 const GURL& url,
36 const security_interstitials::MetricsHelper::ReportDetails settings, 36 const security_interstitials::MetricsHelper::ReportDetails settings,
37 const std::string& sampling_event_name); 37 const std::string& sampling_event_name);
38 ~ChromeMetricsHelper() override; 38 ~ChromeMetricsHelper() override;
(...skipping 18 matching lines...) Expand all
57 std::unique_ptr<extensions::ExperienceSamplingEvent> sampling_event_; 57 std::unique_ptr<extensions::ExperienceSamplingEvent> sampling_event_;
58 #endif 58 #endif
59 #if BUILDFLAG(ENABLE_CAPTIVE_PORTAL_DETECTION) 59 #if BUILDFLAG(ENABLE_CAPTIVE_PORTAL_DETECTION)
60 std::unique_ptr<CaptivePortalMetricsRecorder> captive_portal_recorder_; 60 std::unique_ptr<CaptivePortalMetricsRecorder> captive_portal_recorder_;
61 #endif 61 #endif
62 62
63 DISALLOW_COPY_AND_ASSIGN(ChromeMetricsHelper); 63 DISALLOW_COPY_AND_ASSIGN(ChromeMetricsHelper);
64 }; 64 };
65 65
66 #endif // CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_ 66 #endif // CHROME_BROWSER_INTERSTITIALS_CHROME_METRICS_HELPER_H_
OLDNEW
« no previous file with comments | « chrome/browser/interstitials/chrome_controller_client.cc ('k') | chrome/browser/interstitials/chrome_metrics_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698