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

Unified Diff: chrome/browser/chromeos/policy/device_local_account.h

Issue 2691063002: arc: Use AndroidKioskAppInfoProto.display_name() as ARC++ Kiosk app name. (Closed)
Patch Set: fix post-merge nits from http://crrev.com/2451793002 Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/policy/device_local_account.h
diff --git a/chrome/browser/chromeos/policy/device_local_account.h b/chrome/browser/chromeos/policy/device_local_account.h
index 1613ada1ac8a57db4aabd043fc198c16098deb76..bc7b59adf34c1fac457d255a892be0300f730918 100644
--- a/chrome/browser/chromeos/policy/device_local_account.h
+++ b/chrome/browser/chromeos/policy/device_local_account.h
@@ -18,7 +18,8 @@ namespace policy {
struct ArcKioskAppBasicInfo {
ArcKioskAppBasicInfo(const std::string& package_name,
const std::string& class_name,
- const std::string& action);
+ const std::string& action,
+ const std::string& display_name);
achuithb 2017/02/16 13:41:46 Should display_name be the first arg?
Sergey Poromov 2017/02/16 14:28:28 I think not. It's the latest added value, optional
achuithb 2017/02/16 14:30:51 Acknowledged.
ArcKioskAppBasicInfo(const ArcKioskAppBasicInfo& other);
ArcKioskAppBasicInfo();
~ArcKioskAppBasicInfo();
@@ -28,11 +29,13 @@ struct ArcKioskAppBasicInfo {
const std::string& package_name() const { return package_name_; }
const std::string& class_name() const { return class_name_; }
const std::string& action() const { return action_; }
+ const std::string& display_name() const { return display_name_; }
private:
std::string package_name_;
std::string class_name_;
std::string action_;
+ std::string display_name_;
};
// This must match DeviceLocalAccountInfoProto.AccountType in

Powered by Google App Engine
This is Rietveld 408576698