Move permission warning message handling from PermissionSet to PermissionMessageProvider.

chrome/common/extensions/permissions/extension_permission_message_provider.h

Index: chrome/common/extensions/permissions/extension_permission_message_provider.h
diff --git a/chrome/common/extensions/permissions/extension_permission_message_provider.h b/chrome/common/extensions/permissions/extension_permission_message_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..d684945249269422c9889347a50c8f0bd2d2c601
--- /dev/null
+++ b/chrome/common/extensions/permissions/extension_permission_message_provider.h
@@ -0,0 +1,61 @@
+// Copyright 2013 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_COMMON_EXTENSIONS_PERMISSIONS_EXTENSION_PERMISSION_MESSAGE_PROVIDER_H_
+#define CHROME_COMMON_EXTENSIONS_PERMISSIONS_EXTENSION_PERMISSION_MESSAGE_PROVIDER_H_
+
+#include <set>
+
+#include "base/basictypes.h"
+#include "base/gtest_prod_util.h"
+#include "base/strings/string16.h"
+#include "extensions/common/permissions/permission_message_provider.h"
+
+namespace extensions {
+
+class ExtensionPermissionMessageProvider : public PermissionMessageProvider {

[Matt Perry, 2013/10/16 20:07:04]

Since this is used for apps as well, maybe BasePermissionMessageProvider is more
fitting.

[Yoyo Zhou, 2013/10/17 19:45:52]

Done.

+ public:
+  ExtensionPermissionMessageProvider();
+  virtual ~ExtensionPermissionMessageProvider();
+
+  // PermissionMessageProvider implementation.
+  virtual PermissionMessages GetPermissionMessages(
+      const PermissionSet* permissions) const OVERRIDE;
+  virtual std::vector<string16> GetWarningMessages(
+      const PermissionSet* permissions) const OVERRIDE;
+  virtual std::vector<string16> GetWarningMessagesDetails(
+      const PermissionSet* permissions) const OVERRIDE;
+  virtual bool IsPrivilegeIncrease(
+      const PermissionSet* old_permissions,
+      const PermissionSet* new_permissions) const OVERRIDE;
+
+ private:
+  FRIEND_TEST_ALL_PREFIXES(PermissionsTest, IsHostPrivilegeIncrease);
+
+  // Gets the permission messages for the API permissions.
+  virtual std::set<PermissionMessage> GetAPIPermissionMessages(
+      const PermissionSet* permissions) const;
+
+  // Gets the permission messages for the host permissions.
+  virtual std::set<PermissionMessage> GetHostPermissionMessages(
+      const PermissionSet* permissions) const;
+
+  // Returns true if |new_permissions| has an elevated API privilege level
+  // compared to |old_permissions|.
+  virtual bool IsAPIPrivilegeIncrease(
+      const PermissionSet* old_permissions,
+      const PermissionSet* new_permissions) const;
+
+  // Returns true if |new_permissions| has more host permissions compared to
+  // |old_permissions|.
+  virtual bool IsHostPrivilegeIncrease(
+      const PermissionSet* old_permissions,
+      const PermissionSet* new_permissions) const;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionPermissionMessageProvider);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_COMMON_EXTENSIONS_PERMISSIONS_EXTENSION_PERMISSION_MESSAGE_PROVIDER_H_