Reinstate evil hack to suppress permission warnings for searchProvider

chrome/common/extensions/permissions/chrome_permission_message_provider_unittest.cc

 // Copyright 2014 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.
 
 #include "chrome/common/extensions/permissions/chrome_permission_message_provider.h"
 
 #include <memory>
 #include <vector>
 
 #include "base/macros.h"
@@ skipping 111 matching lines @@
       l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_WARNING_HISTORY_READ),
       message0.message());
   EXPECT_TRUE(message0.submessages().empty());
   const PermissionMessage& message1 = *it++;
   EXPECT_EQ(
       l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_WARNING_USB_DEVICE_LIST),
       message1.message());
   EXPECT_FALSE(message1.submessages().empty());
 }
 
+// Anti-test: Check that adding a parameter to a SettingsOverridePermission
+// doesn't trigger a privilege increase. This is because prior to M46 beta, we
+// failed to store the parameter in the granted_permissions pref. Now we do, and
+// we don't want to bother every user with a spurious permissions warning.
+// See crbug.com/533086 and crbug.com/619759.
+// TODO(treib,devlin): Remove this for M56, when hopefully all users will have
+// updated prefs.
+TEST_F(ChromePermissionMessageProviderUnittest,
+       EvilHackToSuppressSettingsOverrideParameter) {
+  const APIPermissionInfo* info =
+      PermissionsInfo::GetInstance()->GetByID(APIPermission::kSearchProvider);
+
+  APIPermissionSet granted_permissions;
+  granted_permissions.insert(new SettingsOverrideAPIPermission(info));
+
+  APIPermissionSet actual_permissions;
+  actual_permissions.insert(new SettingsOverrideAPIPermission(info, "a.com"));
+
+  EXPECT_FALSE(IsPrivilegeIncrease(granted_permissions, actual_permissions));
+
+  // Just to be safe: Adding the permission (with or without parameter) should
+  // still be considered a privilege escalation.
+  EXPECT_TRUE(IsPrivilegeIncrease(APIPermissionSet(), granted_permissions));
+  EXPECT_TRUE(IsPrivilegeIncrease(APIPermissionSet(), actual_permissions));
+}
+
 }  // namespace extensions