Use ContentSetting in chrome/ instead of PermissionStatus

chrome/browser/permissions/permission_manager.cc

 // Copyright 2015 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_manager.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/callback.h"
 #include "base/memory/ptr_util.h"
 #include "build/build_config.h"
 #include "chrome/browser/background_sync/background_sync_permission_context.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/media/midi_permission_context.h"
 #include "chrome/browser/media/webrtc/media_stream_device_permission_context.h"
 #include "chrome/browser/notifications/notification_permission_context.h"
 #include "chrome/browser/permissions/permission_context_base.h"
 #include "chrome/browser/permissions/permission_manager_factory.h"
 #include "chrome/browser/permissions/permission_request_id.h"
+#include "chrome/browser/permissions/permission_result.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/storage/durable_storage_permission_context.h"
 #include "chrome/browser/tab_contents/tab_util.h"
 #include "chrome/common/features.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/permission_type.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
@@ skipping 16 matching lines @@
 
 using blink::mojom::PermissionStatus;
 using content::PermissionType;
 
 namespace {
 
 // Helper method to convert ContentSetting to PermissionStatus.
 PermissionStatus ContentSettingToPermissionStatus(ContentSetting setting) {
   switch (setting) {
     case CONTENT_SETTING_ALLOW:
-    case CONTENT_SETTING_SESSION_ONLY:
       return PermissionStatus::GRANTED;
     case CONTENT_SETTING_BLOCK:
       return PermissionStatus::DENIED;
     case CONTENT_SETTING_ASK:
       return PermissionStatus::ASK;
+    case CONTENT_SETTING_SESSION_ONLY:
     case CONTENT_SETTING_DETECT_IMPORTANT_CONTENT:
     case CONTENT_SETTING_DEFAULT:
     case CONTENT_SETTING_NUM_SETTINGS:
       break;
   }
 
   NOTREACHED();
   return PermissionStatus::DENIED;
 }
 
-// Wrap a callback taking a PermissionStatus to pass it as a callback taking a
-// ContentSetting.
-void ContentSettingToPermissionStatusCallbackWrapper(
-    const base::Callback<void(PermissionStatus)>& callback,
-    ContentSetting setting) {
-  callback.Run(ContentSettingToPermissionStatus(setting));
-}
-
 // Helper method to convert PermissionType to ContentSettingType.
 ContentSettingsType PermissionTypeToContentSetting(PermissionType permission) {
   switch (permission) {
     case PermissionType::MIDI:
       return CONTENT_SETTINGS_TYPE_MIDI;
     case PermissionType::MIDI_SYSEX:
       return CONTENT_SETTINGS_TYPE_MIDI_SYSEX;
     case PermissionType::PUSH_MESSAGING:
       return CONTENT_SETTINGS_TYPE_PUSH_MESSAGING;
     case PermissionType::NOTIFICATIONS:
@@ skipping 20 matching lines @@
     case PermissionType::NUM:
       // This will hit the NOTREACHED below.
       break;
   }
 
   NOTREACHED() << "Unknown content setting for permission "
                << static_cast<int>(permission);
   return CONTENT_SETTINGS_TYPE_DEFAULT;
 }
 
-// Returns whether the permission has a constant PermissionStatus value (i.e.
+// Returns whether the permission has a constant ContentSetting value (i.e.
 // always approved or always denied)
 // The ContentSettingsTypes for which true is returned will also return nullptr
 // in PermissionManager::GetPermissionContext since they don't have a context.
 bool IsConstantPermission(ContentSettingsType type) {
   return type == CONTENT_SETTINGS_TYPE_MIDI;
 }
 
-void PermissionRequestResponseCallbackWrapper(
+void SubscriptionCallbackWrapper(
     const base::Callback<void(PermissionStatus)>& callback,
-    const std::vector<PermissionStatus>& vector) {
-  DCHECK_EQ(vector.size(), 1ul);
+    ContentSetting content_setting) {
+  callback.Run(ContentSettingToPermissionStatus(content_setting));
+}
+
+void PermissionStatusCallbackWrapper(
+    const base::Callback<void(PermissionStatus)>& callback,
+    const std::vector<ContentSetting>& vector) {
+  DCHECK_EQ(1ul, vector.size());
+  callback.Run(ContentSettingToPermissionStatus(vector[0]));
+}
+
+void PermissionStatusVectorCallbackWrapper(
+    const base::Callback<void(const std::vector<PermissionStatus>&)>& callback,
+    const std::vector<ContentSetting>& content_settings) {
+  std::vector<PermissionStatus> permission_statuses;
+  std::transform(content_settings.begin(), content_settings.end(),
+                 back_inserter(permission_statuses),
+                 ContentSettingToPermissionStatus);
+  callback.Run(permission_statuses);
+}
+
+void ContentSettingCallbackWraper(
+    const base::Callback<void(ContentSetting)>& callback,
+    const std::vector<ContentSetting>& vector) {
+  DCHECK_EQ(1ul, vector.size());
   callback.Run(vector[0]);
 }
 
 // Function used for handling permission types which do not change their
 // value i.e. they are always approved or always denied etc.
 // CONTENT_SETTING_DEFAULT is returned if the permission needs further handling.
 // This function should only be called when IsConstantPermission has returned
 // true for the PermissionType.
-blink::mojom::PermissionStatus GetPermissionStatusForConstantPermission(
+ContentSetting GetContentSettingForConstantPermission(
     ContentSettingsType type) {
   DCHECK(IsConstantPermission(type));
   switch (type) {
     case CONTENT_SETTINGS_TYPE_MIDI:
-      return PermissionStatus::GRANTED;
+      return CONTENT_SETTING_ALLOW;
     default:
-      return PermissionStatus::DENIED;
+      return CONTENT_SETTING_BLOCK;
   }
 }
 
 }  // anonymous namespace
 
 class PermissionManager::PendingRequest {
  public:
   PendingRequest(
       content::RenderFrameHost* render_frame_host,
       const std::vector<ContentSettingsType>& permissions,
-      const base::Callback<void(const std::vector<PermissionStatus>&)>&
-          callback)
+      const base::Callback<void(const std::vector<ContentSetting>&)>& callback)
       : render_process_id_(render_frame_host->GetProcess()->GetID()),
         render_frame_id_(render_frame_host->GetRoutingID()),
         callback_(callback),
         permissions_(permissions),
-        results_(permissions.size(), PermissionStatus::DENIED),
+        results_(permissions.size(), CONTENT_SETTING_BLOCK),
         remaining_results_(permissions.size()) {}
 
-  void SetPermissionStatus(int permission_id, PermissionStatus status) {
+  void SetContentSetting(int permission_id, ContentSetting content_setting) {
     DCHECK(!IsComplete());
 
-    results_[permission_id] = status;
+    results_[permission_id] = content_setting;
     --remaining_results_;
   }
 
   bool IsComplete() const {
     return remaining_results_ == 0;
   }
 
   int render_process_id() const { return render_process_id_; }
   int render_frame_id() const { return render_frame_id_; }
 
-  const base::Callback<void(const std::vector<PermissionStatus>&)>
-  callback() const {
+  const base::Callback<void(const std::vector<ContentSetting>&)> callback()
+      const {
     return callback_;
   }
 
   std::vector<ContentSettingsType> permissions() const {
     return permissions_;
   }
 
-  std::vector<PermissionStatus> results() const {
-    return results_;
-  }
+  std::vector<ContentSetting> results() const { return results_; }
 
  private:
   int render_process_id_;
   int render_frame_id_;
-  const base::Callback<void(const std::vector<PermissionStatus>&)> callback_;
+  const base::Callback<void(const std::vector<ContentSetting>&)> callback_;
   std::vector<ContentSettingsType> permissions_;
-  std::vector<PermissionStatus> results_;
+  std::vector<ContentSetting> results_;
   size_t remaining_results_;
 };
 
 struct PermissionManager::Subscription {
   ContentSettingsType permission;
   GURL requesting_origin;
   GURL embedding_origin;
-  base::Callback<void(PermissionStatus)> callback;
-  PermissionStatus current_value;
+  base::Callback<void(ContentSetting)> callback;
+  ContentSetting current_value;
 };
 
 // static
 PermissionManager* PermissionManager::Get(Profile* profile) {
   return PermissionManagerFactory::GetForProfile(profile);
 }
 
 PermissionManager::PermissionManager(Profile* profile)
     : profile_(profile),
       weak_ptr_factory_(this) {
@@ skipping 36 matching lines @@
   if (!subscriptions_.IsEmpty())
     HostContentSettingsMapFactory::GetForProfile(profile_)
         ->RemoveObserver(this);
 }
 
 int PermissionManager::RequestPermission(
     ContentSettingsType content_settings_type,
     content::RenderFrameHost* render_frame_host,
     const GURL& requesting_origin,
     bool user_gesture,
-    const base::Callback<void(PermissionStatus)>& callback) {
+    const base::Callback<void(ContentSetting)>& callback) {
   return RequestPermissions(
       std::vector<ContentSettingsType>(1, content_settings_type),
       render_frame_host, requesting_origin, user_gesture,
-      base::Bind(&PermissionRequestResponseCallbackWrapper, callback));
+      base::Bind(&ContentSettingCallbackWraper, callback));
 }
 
 int PermissionManager::RequestPermissions(
     const std::vector<ContentSettingsType>& permissions,
     content::RenderFrameHost* render_frame_host,
     const GURL& requesting_origin,
     bool user_gesture,
-    const base::Callback<void(const std::vector<PermissionStatus>&)>&
-        callback) {
+    const base::Callback<void(const std::vector<ContentSetting>&)>& callback) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (permissions.empty()) {
-    callback.Run(std::vector<PermissionStatus>());
+    callback.Run(std::vector<ContentSetting>());
     return kNoPendingOperation;
   }
 
   content::WebContents* web_contents =
       content::WebContents::FromRenderFrameHost(render_frame_host);
   GURL embedding_origin = web_contents->GetLastCommittedURL().GetOrigin();
 
   int request_id = pending_requests_.Add(base::MakeUnique<PendingRequest>(
       render_frame_host, permissions, callback));
 
   const PermissionRequestID request(render_frame_host, request_id);
 
   for (size_t i = 0; i < permissions.size(); ++i) {
     const ContentSettingsType permission = permissions[i];
 
     if (IsConstantPermission(permission) || !GetPermissionContext(permission)) {
       // Track permission request usages even for constant permissions.
       PermissionUmaUtil::PermissionRequested(permission, requesting_origin,
                                              embedding_origin, profile_);
       OnPermissionsRequestResponseStatus(
-          request_id, i, GetPermissionStatusForConstantPermission(permission));
+          request_id, i, GetContentSettingForConstantPermission(permission));
       continue;
     }
 
     PermissionContextBase* context = GetPermissionContext(permission);
     context->RequestPermission(
         web_contents, request, requesting_origin, user_gesture,
-        base::Bind(&ContentSettingToPermissionStatusCallbackWrapper,
-            base::Bind(&PermissionManager::OnPermissionsRequestResponseStatus,
-                weak_ptr_factory_.GetWeakPtr(), request_id, i)));
+        base::Bind(&PermissionManager::OnPermissionsRequestResponseStatus,
+                   weak_ptr_factory_.GetWeakPtr(), request_id, i));
   }
 
   // The request might have been resolved already.
   if (!pending_requests_.Lookup(request_id))
     return kNoPendingOperation;
 
   return request_id;
 }
 
-PermissionStatus PermissionManager::GetPermissionStatus(
+PermissionResult PermissionManager::GetPermissionStatus(
     ContentSettingsType permission,
     const GURL& requesting_origin,
     const GURL& embedding_origin) {
-  if (IsConstantPermission(permission))
-    return GetPermissionStatusForConstantPermission(permission);
+  if (IsConstantPermission(permission)) {
+    return PermissionResult(GetContentSettingForConstantPermission(permission),
+                            PermissionStatusSource::UNSPECIFIED);
+  }
   PermissionContextBase* context = GetPermissionContext(permission);
-  return ContentSettingToPermissionStatus(
-      context
-          ->GetPermissionStatus(requesting_origin.GetOrigin(),
-                                embedding_origin.GetOrigin())
-          .content_setting);
+  PermissionResult result = context->GetPermissionStatus(
+      requesting_origin.GetOrigin(), embedding_origin.GetOrigin());
+  DCHECK(result.content_setting == CONTENT_SETTING_ALLOW ||
+         result.content_setting == CONTENT_SETTING_ASK ||
+         result.content_setting == CONTENT_SETTING_BLOCK);
+  return result;
 }
 
 int PermissionManager::RequestPermission(
     PermissionType permission,
     content::RenderFrameHost* render_frame_host,
     const GURL& requesting_origin,
     bool user_gesture,
     const base::Callback<void(PermissionStatus)>& callback) {
   ContentSettingsType content_settings_type =
       PermissionTypeToContentSetting(permission);
   return RequestPermissions(
       std::vector<ContentSettingsType>(1, content_settings_type),
       render_frame_host, requesting_origin, user_gesture,
-      base::Bind(&PermissionRequestResponseCallbackWrapper, callback));
+      base::Bind(&PermissionStatusCallbackWrapper, callback));
 }
 
 int PermissionManager::RequestPermissions(
     const std::vector<PermissionType>& permissions,
     content::RenderFrameHost* render_frame_host,
     const GURL& requesting_origin,
     bool user_gesture,
     const base::Callback<void(const std::vector<PermissionStatus>&)>&
         callback) {
   std::vector<ContentSettingsType> content_settings_types;
   std::transform(permissions.begin(), permissions.end(),
                  back_inserter(content_settings_types),
                  PermissionTypeToContentSetting);
-  return RequestPermissions(content_settings_types, render_frame_host,
-                            requesting_origin, user_gesture, callback);
+  return RequestPermissions(
+      content_settings_types, render_frame_host, requesting_origin,
+      user_gesture,
+      base::Bind(&PermissionStatusVectorCallbackWrapper, callback));
 }
 
 PermissionContextBase* PermissionManager::GetPermissionContext(
     ContentSettingsType type) {
   const auto& it = permission_contexts_.find(type);
   return it == permission_contexts_.end() ? nullptr : it->second.get();
 }
 
 void PermissionManager::OnPermissionsRequestResponseStatus(
     int request_id,
     int permission_id,
-    PermissionStatus status) {
+    ContentSetting content_setting) {
   PendingRequest* pending_request = pending_requests_.Lookup(request_id);
-  pending_request->SetPermissionStatus(permission_id, status);
+  pending_request->SetContentSetting(permission_id, content_setting);
 
   if (!pending_request->IsComplete())
     return;
 
   pending_request->callback().Run(pending_request->results());
   pending_requests_.Remove(request_id);
 }
 
 void PermissionManager::CancelPermissionRequest(int request_id) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@@ skipping 28 matching lines @@
 
   context->ResetPermission(requesting_origin.GetOrigin(),
                            embedding_origin.GetOrigin());
 }
 
 PermissionStatus PermissionManager::GetPermissionStatus(
     PermissionType permission,
     const GURL& requesting_origin,
     const GURL& embedding_origin) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  return GetPermissionStatus(PermissionTypeToContentSetting(permission),
-                             requesting_origin, embedding_origin);
+  return ContentSettingToPermissionStatus(
+      GetPermissionStatus(PermissionTypeToContentSetting(permission),
+                          requesting_origin, embedding_origin)
+          .content_setting);
 }
 
 int PermissionManager::SubscribePermissionStatusChange(
     PermissionType permission,
     const GURL& requesting_origin,
     const GURL& embedding_origin,
     const base::Callback<void(PermissionStatus)>& callback) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if (subscriptions_.IsEmpty())
     HostContentSettingsMapFactory::GetForProfile(profile_)->AddObserver(this);
 
   ContentSettingsType content_type = PermissionTypeToContentSetting(permission);
   auto subscription = base::MakeUnique<Subscription>();
   subscription->permission = content_type;
   subscription->requesting_origin = requesting_origin;
   subscription->embedding_origin = embedding_origin;
-  subscription->callback = callback;
+  subscription->callback = base::Bind(&SubscriptionCallbackWrapper, callback);
 
   subscription->current_value =
-      GetPermissionStatus(content_type, requesting_origin, embedding_origin);
+      GetPermissionStatus(content_type, requesting_origin, embedding_origin)
+          .content_setting;
 
   return subscriptions_.Add(std::move(subscription));
 }
 
 void PermissionManager::UnsubscribePermissionStatusChange(int subscription_id) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   // Whether |subscription_id| is known will be checked by the Remove() call.
   subscriptions_.Remove(subscription_id);
 
   if (subscriptions_.IsEmpty())
@@ skipping 23 matching lines @@
     if (subscription->permission != content_type)
       continue;
 
     if (primary_pattern.IsValid() &&
         !primary_pattern.Matches(subscription->requesting_origin))
       continue;
     if (secondary_pattern.IsValid() &&
         !secondary_pattern.Matches(subscription->embedding_origin))
       continue;
 
-    PermissionStatus new_value = GetPermissionStatus(
-        subscription->permission, subscription->requesting_origin,
-        subscription->embedding_origin);
+    ContentSetting new_value =
+        GetPermissionStatus(subscription->permission,
+                            subscription->requesting_origin,
+                            subscription->embedding_origin)
+            .content_setting;
     if (subscription->current_value == new_value)
       continue;
 
     subscription->current_value = new_value;
 
     // Add the callback to |callbacks| which will be run after the loop to
     // prevent re-entrance issues.
     callbacks.push_back(base::Bind(subscription->callback, new_value));
   }
 
   for (const auto& callback : callbacks)
     callback.Run();
 }