Content Settings: Add RevocationObserver to measure when permissions are revoked

chrome/browser/content_settings/host_content_settings_map_unittest.cc

 // Copyright (c) 2012 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 "base/auto_reset.h"
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/content_settings/content_settings_mock_observer.h"
@@ skipping 1052 matching lines @@
   host_content_settings_map->AddObserver(&mock_observer);
 
   EXPECT_EQ(CONTENT_SETTING_ALLOW,
             host_content_settings_map->GetContentSetting(
                 host, host, CONTENT_SETTINGS_TYPE_IMAGES, std::string()));
   host_content_settings_map->SetContentSettingDefaultScope(
       host, GURL(), CONTENT_SETTINGS_TYPE_IMAGES, std::string(),
       CONTENT_SETTING_DEFAULT);
 }
 
+TEST_F(HostContentSettingsMapTest, RevocationObserver) {
+  TestingProfile profile;
+  HostContentSettingsMap* map =
+      HostContentSettingsMapFactory::GetForProfile(&profile);
+  content_settings::MockObserver mock_observer;
+  map->AddRevocationObserver(&mock_observer);
+
+  GURL host("http://example.com/");
+
+  ContentSettingsPattern pattern =
+      ContentSettingsPattern::FromURLNoWildcard(host);
+
+  // Allow->Block triggers a revocation.
+
+  EXPECT_CALL(mock_observer,
+              OnContentSettingRevoked(host, host,
+                                      CONTENT_SETTINGS_TYPE_GEOLOCATION, ""));
+
+  map->SetContentSettingDefaultScope(
+      host, host, CONTENT_SETTINGS_TYPE_GEOLOCATION, "", CONTENT_SETTING_ALLOW);
+
+  map->SetContentSettingDefaultScope(
+      host, host, CONTENT_SETTINGS_TYPE_GEOLOCATION, "", CONTENT_SETTING_BLOCK);
+
+  ::testing::Mock::VerifyAndClearExpectations(&mock_observer);
+
+  // Block->Allow does not trigger a revocation.
+  EXPECT_CALL(mock_observer,
+              OnContentSettingRevoked(host, host,
+                                      CONTENT_SETTINGS_TYPE_GEOLOCATION, ""))
+      .Times(0);
+
+  map->SetContentSettingDefaultScope(
+      host, host, CONTENT_SETTINGS_TYPE_GEOLOCATION, "", CONTENT_SETTING_ALLOW);
+
+  ::testing::Mock::VerifyAndClearExpectations(&mock_observer);
+
+  // Allow->Default triggers a revocation when default is 'ask'.

[kcarattini, 2016/03/17 02:51:22]

There's no way to clear or reset all permissions from the settings menu at the
moment, but I know people are interested in making that happen. If it does,
we'll need to think about whether resetting all permissions should count as a
revocation.

[tsergeant, 2016/03/17 06:25:40]

Hmm, good point. Under this system the 'Clear & Reset' button in the Android
Site Settings page will count as revoking each individual permission, which I
think is a good thing. I don't think we would want revoking every single
permission on the device to count, though.

It will also depend on the implementation of that button as to whether this code
gets called.

+  EXPECT_CALL(mock_observer,
+              OnContentSettingRevoked(host, host,
+                                      CONTENT_SETTINGS_TYPE_GEOLOCATION, ""));
+
+  map->SetDefaultContentSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION,
+                                CONTENT_SETTING_ASK);
+
+  map->SetContentSettingDefaultScope(host, host,
+                                     CONTENT_SETTINGS_TYPE_GEOLOCATION, "",
+                                     CONTENT_SETTING_DEFAULT);
+
+  // Allow->Default does not trigger a revocation when default is 'allow'.
+  EXPECT_CALL(mock_observer,
+              OnContentSettingRevoked(host, host,
+                                      CONTENT_SETTINGS_TYPE_GEOLOCATION, ""))
+      .Times(0);
+
+  map->SetDefaultContentSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION,
+                                CONTENT_SETTING_ALLOW);
+
+  map->SetContentSettingDefaultScope(host, host,
+                                     CONTENT_SETTINGS_TYPE_GEOLOCATION, "",
+                                     CONTENT_SETTING_ALLOW);
+  map->SetContentSettingDefaultScope(host, host,
+                                     CONTENT_SETTINGS_TYPE_GEOLOCATION, "",
+                                     CONTENT_SETTING_DEFAULT);
+
+  ::testing::Mock::VerifyAndClearExpectations(&mock_observer);
+}
+
 TEST_F(HostContentSettingsMapTest, GuestProfile) {
   TestingProfile profile;
   profile.SetGuestSession(true);
   HostContentSettingsMap* host_content_settings_map =
       HostContentSettingsMapFactory::GetForProfile(&profile);
 
   GURL host("http://example.com/");
   ContentSettingsPattern pattern =
       ContentSettingsPattern::FromString("[*.]example.com");
 
@@ skipping 51 matching lines @@
   profile.GetPrefs()->Set(GetPrefName(CONTENT_SETTINGS_TYPE_IMAGES), *value);
 
   // Test that during construction all the prefs get cleared.
   HostContentSettingsMapFactory::GetForProfile(&profile);
 
   const base::DictionaryValue* all_settings_dictionary =
       profile.GetPrefs()->GetDictionary(
           GetPrefName(CONTENT_SETTINGS_TYPE_IMAGES));
   EXPECT_TRUE(all_settings_dictionary->empty());
 }