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

Side by Side Diff: components/metrics/metrics_service_client.h

Issue 573403002: Change UMA proto product field to be an int32. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_ 5 #ifndef COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_
6 #define COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_ 6 #define COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 12 matching lines...) Expand all
23 public: 23 public:
24 virtual ~MetricsServiceClient() {} 24 virtual ~MetricsServiceClient() {}
25 25
26 // Registers the client id with other services (e.g. crash reporting), called 26 // Registers the client id with other services (e.g. crash reporting), called
27 // when metrics recording gets enabled. 27 // when metrics recording gets enabled.
28 virtual void SetMetricsClientId(const std::string& client_id) = 0; 28 virtual void SetMetricsClientId(const std::string& client_id) = 0;
29 29
30 // Whether there's an "off the record" (aka "Incognito") session active. 30 // Whether there's an "off the record" (aka "Incognito") session active.
31 virtual bool IsOffTheRecordSessionActive() = 0; 31 virtual bool IsOffTheRecordSessionActive() = 0;
32 32
33 // Returns the product value to use in uploaded reports, which will be used to
34 // set the ChromeUserMetricsExtension.product field. See comments on that
35 // field on why it's an int32 rather than an enum.
36 virtual int32 GetProduct() = 0;
Ilya Sherman 2014/09/25 18:29:55 According to basictypes.h, we should use int32_t i
Alexei Svitkine (slow) 2014/09/26 16:51:09 Done.
37
33 // Returns the current application locale (e.g. "en-US"). 38 // Returns the current application locale (e.g. "en-US").
34 virtual std::string GetApplicationLocale() = 0; 39 virtual std::string GetApplicationLocale() = 0;
35 40
36 // Retrieves the brand code string associated with the install, returning 41 // Retrieves the brand code string associated with the install, returning
37 // false if no brand code is available. 42 // false if no brand code is available.
38 virtual bool GetBrand(std::string* brand_code) = 0; 43 virtual bool GetBrand(std::string* brand_code) = 0;
39 44
40 // Returns the release channel (e.g. stable, beta, etc) of the application. 45 // Returns the release channel (e.g. stable, beta, etc) of the application.
41 virtual SystemProfileProto::Channel GetChannel() = 0; 46 virtual SystemProfileProto::Channel GetChannel() = 0;
42 47
(...skipping 20 matching lines...) Expand all
63 const base::Callback<void(int)>& on_upload_complete) = 0; 68 const base::Callback<void(int)>& on_upload_complete) = 0;
64 69
65 // Returns the name of a key under HKEY_CURRENT_USER that can be used to store 70 // Returns the name of a key under HKEY_CURRENT_USER that can be used to store
66 // backups of metrics data. Unused except on Windows. 71 // backups of metrics data. Unused except on Windows.
67 virtual base::string16 GetRegistryBackupKey(); 72 virtual base::string16 GetRegistryBackupKey();
68 }; 73 };
69 74
70 } // namespace metrics 75 } // namespace metrics
71 76
72 #endif // COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_ 77 #endif // COMPONENTS_METRICS_METRICS_SERVICE_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698