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

Unified Diff: chrome/common/extensions/permissions/chrome_permission_message_provider.h

Issue 27446002: Move permission warning message handling from PermissionSet to PermissionMessageProvider. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/permissions/chrome_permission_message_provider.h
diff --git a/chrome/common/extensions/permissions/chrome_permission_message_provider.h b/chrome/common/extensions/permissions/chrome_permission_message_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..05125f43763d156431702e5370e712d84931bbf6
--- /dev/null
+++ b/chrome/common/extensions/permissions/chrome_permission_message_provider.h
@@ -0,0 +1,65 @@
+// 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_CHROME_PERMISSION_MESSAGE_PROVIDER_H_
+#define CHROME_COMMON_EXTENSIONS_PERMISSIONS_CHROME_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 ChromePermissionMessageProvider : public PermissionMessageProvider {
+ public:
+ ChromePermissionMessageProvider();
+ virtual ~ChromePermissionMessageProvider();
+
+ // PermissionMessageProvider implementation.
+ virtual PermissionMessages GetPermissionMessages(
+ const PermissionSet* permissions,
+ Manifest::Type extension_type) const OVERRIDE;
+ virtual std::vector<string16> GetWarningMessages(
+ const PermissionSet* permissions,
+ Manifest::Type extension_type) const OVERRIDE;
+ virtual std::vector<string16> GetWarningMessagesDetails(
+ const PermissionSet* permissions,
+ Manifest::Type extension_type) const OVERRIDE;
+ virtual bool IsPrivilegeIncrease(
+ const PermissionSet* old_permissions,
+ const PermissionSet* new_permissions,
+ Manifest::Type extension_type) const OVERRIDE;
+
+ private:
+ // Gets the permission messages for the API permissions.
+ std::set<PermissionMessage> GetAPIPermissionMessages(
+ const PermissionSet* permissions) const;
+
+ // Gets the permission messages for the host permissions.
+ std::set<PermissionMessage> GetHostPermissionMessages(
+ const PermissionSet* permissions,
+ Manifest::Type extension_type) const;
+
+ // Returns true if |new_permissions| has an elevated API privilege level
+ // compared to |old_permissions|.
+ bool IsAPIPrivilegeIncrease(
+ const PermissionSet* old_permissions,
+ const PermissionSet* new_permissions) const;
+
+ // Returns true if |new_permissions| has more host permissions compared to
+ // |old_permissions|.
+ bool IsHostPrivilegeIncrease(
+ const PermissionSet* old_permissions,
+ const PermissionSet* new_permissions,
+ Manifest::Type extension_type) const;
+
+ DISALLOW_COPY_AND_ASSIGN(ChromePermissionMessageProvider);
+};
+
+} // namespace extensions
+
+#endif // CHROME_COMMON_EXTENSIONS_PERMISSIONS_CHROME_PERMISSION_MESSAGE_PROVIDER_H_

Powered by Google App Engine
This is Rietveld 408576698