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

Unified Diff: chrome/browser/extensions/standard_management_policy_provider_unittest.cc

Issue 384423002: [Canceled] Extensions: Add install_flags parameter to ManagementPolicy::UserMayLoad (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update tests Created 6 years, 5 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/extensions/standard_management_policy_provider_unittest.cc
diff --git a/chrome/browser/extensions/standard_management_policy_provider_unittest.cc b/chrome/browser/extensions/standard_management_policy_provider_unittest.cc
index 34c1969b27cec93b52f78627dc27779f24edf392..fb67e1145f2d8a9320fe6b352eee10a720a6fcb8 100644
--- a/chrome/browser/extensions/standard_management_policy_provider_unittest.cc
+++ b/chrome/browser/extensions/standard_management_policy_provider_unittest.cc
@@ -10,9 +10,12 @@
#include "chrome/browser/extensions/test_extension_prefs.h"
#include "content/public/test/test_browser_thread.h"
#include "extensions/browser/extension_prefs.h"
+#include "extensions/browser/install_flag.h"
#include "extensions/common/manifest_constants.h"
#include "testing/gtest/include/gtest/gtest.h"
+const int kInstallFlags = extensions::kInstallFlagNone;
+
namespace extensions {
class StandardManagementPolicyProviderTest : public testing::Test {
@@ -56,7 +59,7 @@ TEST_F(StandardManagementPolicyProviderTest, RequiredExtension) {
CreateExtension(Manifest::EXTERNAL_POLICY_DOWNLOAD, true);
base::string16 error16;
- EXPECT_TRUE(provider_.UserMayLoad(extension.get(), &error16));
+ EXPECT_TRUE(provider_.UserMayLoad(extension.get(), kInstallFlags, &error16));
EXPECT_EQ(base::string16(), error16);
// We won't check the exact wording of the error, but it should say
@@ -74,7 +77,7 @@ TEST_F(StandardManagementPolicyProviderTest, NotRequiredExtension) {
CreateExtension(Manifest::INTERNAL, false);
base::string16 error16;
- EXPECT_TRUE(provider_.UserMayLoad(extension.get(), &error16));
+ EXPECT_TRUE(provider_.UserMayLoad(extension.get(), kInstallFlags, &error16));
EXPECT_EQ(base::string16(), error16);
EXPECT_TRUE(provider_.UserMayModifySettings(extension.get(), &error16));
EXPECT_EQ(base::string16(), error16);
« no previous file with comments | « chrome/browser/extensions/standard_management_policy_provider.cc ('k') | chrome/browser/extensions/unpacked_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698