Permissions: Clear embargo if user changes an embargoed permission's setting.

chrome/browser/permissions/permission_decision_auto_blocker_unittest.cc

 // Copyright 2016 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/browser/permissions/permission_decision_auto_blocker.h"
 
 #include <map>
 
 #include "base/bind.h"
 #include "base/run_loop.h"
@@ skipping 141 matching lines @@
   }
 
  private:
   PermissionDecisionAutoBlocker* autoblocker_;
   base::test::ScopedFeatureList feature_list_;
   base::SimpleTestClock* clock_;
   bool last_embargoed_status_;
   bool callback_was_run_;
 };
 
+// Check removing the the emargo for a single permission on a site works, and

[dominickn, 2017/04/03 01:35:31]

sp. embargo

[Patti Lor, 2017/04/05 08:34:27]

Done.

+// that it doesn't interfere with other embargoed permissions or the same
+// permission embargoed on other sites.
+TEST_F(PermissionDecisionAutoBlockerUnitTest, RemoveEmbargoByURL) {

[dominickn, 2017/04/03 01:35:30]

Url

[Patti Lor, 2017/04/05 08:34:27]

Done.

+  GURL url1("https://www.google.com");
+  GURL url2("https://www.example.com");
+
+  // Record dismissals for location and notifications in |url1|.
+  autoblocker()->RecordDismissAndEmbargo(url1,

[dominickn, 2017/04/03 01:35:31]

I'd EXPECT_FALSE / EXPECT_TRUE all of these calls (returns true when it's
actually embargoed).

[Patti Lor, 2017/04/05 08:34:27]

Done.

+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  autoblocker()->RecordDismissAndEmbargo(url1,
+                                         CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  // Record dismissals for location in |url2|.
+  autoblocker()->RecordDismissAndEmbargo(url2,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url2,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url2,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url2,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+
+  // Verify all dismissals recorded above resulted in embargo.
+  PermissionResult result =
+      autoblocker()->GetEmbargoResult(url1, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+  result = autoblocker()->GetEmbargoResult(url1,
+                                           CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+  result =
+      autoblocker()->GetEmbargoResult(url2, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+
+  // Remove the embargo on notifications. Verify it is no longer under embargo,
+  // but location still is.
+  autoblocker()->RemoveEmbargoByURL(url1, CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  result =
+      autoblocker()->GetEmbargoResult(url1, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+  // Notification's default setting is |CONTENT_SETTING_ASK|.
+  result = autoblocker()->GetEmbargoResult(url1,
+                                           CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  EXPECT_EQ(CONTENT_SETTING_ASK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::UNSPECIFIED, result.source);
+  // Verify |url2|'s embargo is still intact as well.
+  result =
+      autoblocker()->GetEmbargoResult(url2, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+}
+
+// Test that removing embargo from blacklisted permissions also works.
+TEST_F(PermissionDecisionAutoBlockerUnitTest,
+       RemoveEmbargoByURLForBlacklistedPermission) {
+  GURL url("https://www.example.com");
+
+  // Place under embargo and verify.
+  PlaceUnderBlacklistEmbargo(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  PermissionResult result =
+      autoblocker()->GetEmbargoResult(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::SAFE_BROWSING_BLACKLIST, result.source);
+
+  // Remove embargo and verify.
+  autoblocker()->RemoveEmbargoByURL(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  result =
+      autoblocker()->GetEmbargoResult(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_ASK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::UNSPECIFIED, result.source);
+}
+
+// Test it still only takes one more dismissal to re-trigger embargo after
+// removing the embargo status for a site.
+TEST_F(PermissionDecisionAutoBlockerUnitTest,
+       DismissAfterRemovingEmbargoByURL) {
+  GURL url("https://www.example.com");
+
+  // Record dismissals for location.
+  autoblocker()->RecordDismissAndEmbargo(url,

[dominickn, 2017/04/03 01:35:31]

EXPECT_TRUE / EXPECT_FALSE these calls

[Patti Lor, 2017/04/05 08:34:27]

Done.

+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  autoblocker()->RecordDismissAndEmbargo(url,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+
+  // Verify location is under embargo.
+  PermissionResult result =
+      autoblocker()->GetEmbargoResult(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+
+  // Remove embargo.
+  autoblocker()->RemoveEmbargoByURL(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+

[dominickn, 2017/04/03 01:35:31]

Nit: verify that it's not under embargo here

[Patti Lor, 2017/04/05 08:34:27]

Done.

+  // Record another dismissal and verify location is under embargo again.
+  autoblocker()->RecordDismissAndEmbargo(url,
+                                         CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  result =
+      autoblocker()->GetEmbargoResult(url, CONTENT_SETTINGS_TYPE_GEOLOCATION);
+  EXPECT_EQ(CONTENT_SETTING_BLOCK, result.content_setting);
+  EXPECT_EQ(PermissionStatusSource::MULTIPLE_DISMISSALS, result.source);
+}
+
 TEST_F(PermissionDecisionAutoBlockerUnitTest, RemoveCountsByUrl) {
   GURL url1("https://www.google.com");
   GURL url2("https://www.example.com");
 
   // Record some dismissals.
   EXPECT_FALSE(autoblocker()->RecordDismissAndEmbargo(
       url1, CONTENT_SETTINGS_TYPE_GEOLOCATION));
   EXPECT_EQ(1, autoblocker()->GetDismissCount(
                    url1, CONTENT_SETTINGS_TYPE_GEOLOCATION));
 
@@ skipping 445 matching lines @@
   db_manager->BlacklistUrlPermissions(url, blacklisted_permissions);
   SetSafeBrowsingDatabaseManagerAndTimeoutForTesting(db_manager,
                                                      0 /* timeout in ms */);
 
   CheckSafeBrowsingBlacklist(url, CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
   EXPECT_FALSE(last_embargoed_status());
   histograms.ExpectUniqueSample(
       "Permissions.AutoBlocker.SafeBrowsingResponse",
       static_cast<int>(SafeBrowsingResponse::NOT_BLACKLISTED), 1);
 }