Added KioskModeAppPack proto.

chrome/browser/policy/app_pack_updater.h

Index: chrome/browser/policy/app_pack_updater.h
diff --git a/chrome/browser/policy/app_pack_updater.h b/chrome/browser/policy/app_pack_updater.h
new file mode 100644
index 0000000000000000000000000000000000000000..741a3c4d92ffc8825008ff874bc540bf529955b7
--- /dev/null
+++ b/chrome/browser/policy/app_pack_updater.h
@@ -0,0 +1,22 @@
+// Copyright (c) 2012 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_POLICY_APP_PACK_UPDATER_H_
+#define CHROME_BROWSER_POLICY_APP_PACK_UPDATER_H_
+#pragma once
+
+namespace policy {
+
+// Work in progress: http://crosbug.com/25463
+class AppPackUpdater {

[pastarmovj, 2012/02/21 14:06:40]

I guess this class is going to grow beyond a container for three constants
right? If so I am fine with how you have it now. If not I guess a namespace
would be better.

Btw this is a rather vague memory but I think there was some issues with static
members and inheritance over lib boundaries. Could that impact mock-ability of
this class? What speaks against having those as a global constants in the policy
namespace possible renamed s/^k/kAppPack/.

[Joao da Silva, 2012/02/21 15:24:49]

It a big class :-) This is the bare minimum for this CL, since the field names
are required to decode the message.
I don't expect these constants to be used out of libbrowser, and I also don't
think they matter to mocking. I've scoped them within the class to avoid
polluting the policy namespace, and to signal that these are AppPack-related
constants. Having an kAppPack prefix in the policy namespace would also work.

+ public:
+  // Keys for the entries in the KioskModeAppPack dictionary policy.
+  static const char kExtensionId[];
+  static const char kUpdateUrl[];
+  static const char kKeyChecksum[];
+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_POLICY_APP_PACK_UPDATER_H_