Delete fullscreen/mouselock pref data.

chrome/browser/ui/webui/settings/site_settings_handler.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/ui/webui/settings/site_settings_handler.h"
 
 #include <algorithm>
 #include <memory>
 #include <string>
 #include <utility>
@@ skipping 206 matching lines @@
 }
 
 void SiteSettingsHandler::OnContentSettingChanged(
     const ContentSettingsPattern& primary_pattern,
     const ContentSettingsPattern& secondary_pattern,
     ContentSettingsType content_type,
     std::string resource_identifier) {
   if (!site_settings::HasRegisteredGroupName(content_type))
     return;
 
+  // These content types are deprecated and should not trigger a UI update.

[Matt Giuca, 2016/10/26 04:18:24]

I wish I didn't have to do this (it isn't strictly necessary), but the problem
is that without this, the test SiteSettingsHandlerTest.Incognito fails:
chrome/browser/ui/webui/settings/site_settings_handler_unittest.cc

Because it uses gmock to expect that the final call is onIncognitoStatusChanged,
and this breaks that assumption.

This is temporary; it will be deleted in a follow-up CL
(https://codereview.chromium.org/2419413002/) when I remove these enum values.

[stevenjb, 2016/10/26 19:31:00]

Could you add a TODO with a bug link? Generally modifying production code for
tests kind of sucks, so let's make sure this is well documented even if it's
going away almost immediately.

[Matt Giuca, 2016/10/26 23:26:47]

Done.

+  if (content_type == CONTENT_SETTINGS_TYPE_FULLSCREEN ||
+      content_type == CONTENT_SETTINGS_TYPE_MOUSELOCK) {
+    return;
+  }
+
   if (primary_pattern.ToString().empty()) {
     CallJavascriptFunction(
         "cr.webUIListenerCallback",
         base::StringValue("contentSettingCategoryChanged"),
         base::StringValue(site_settings::ContentSettingsTypeToGroupName(
             content_type)));
   } else {
     CallJavascriptFunction(
         "cr.webUIListenerCallback",
         base::StringValue("contentSettingSitePermissionChanged"),
@@ skipping 452 matching lines @@
     origin = content::kUnreachableWebDataURL;
   }
 
   content::HostZoomMap* host_zoom_map;
   host_zoom_map = content::HostZoomMap::GetDefaultForBrowserContext(profile_);
   double default_level = host_zoom_map->GetDefaultZoomLevel();
   host_zoom_map->SetZoomLevelForHost(origin, default_level);
 }
 
 }  // namespace settings