Revert: Per navigation sticky media permissions.

chrome/browser/media/media_stream_devices_controller.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 "chrome/browser/media/media_stream_devices_controller.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/content_settings/content_settings_provider.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
 #include "chrome/browser/media/media_stream_capture_indicator.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/content_settings.h"
 #include "chrome/common/content_settings_pattern.h"
 #include "chrome/common/pref_names.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "content/public/browser/browser_thread.h"
-#include "content/public/browser/navigation_entry.h"
 #include "content/public/common/media_stream_request.h"
 #include "extensions/common/constants.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/login/users/user_manager.h"
 #endif
 
 using content::BrowserThread;
 
 namespace {
 
-// This prefix is combined with request security origins to store media access
-// permissions that the user has granted a specific page navigation instance.
-// The string value stored with the navigation instance will contain one or more
-// kMediaPermissionXxx constants that indicates the permission(s) that the user
-// has granted the page.
-const char kMediaPermissionKeyPrefix[] = "media_permissions#";
-const base::char16 kMediaPermissionAudio = static_cast<base::char16>('a');
-const base::char16 kMediaPermissionVideo = static_cast<base::char16>('v');
-
 bool HasAvailableDevicesForRequest(const content::MediaStreamRequest& request) {
   const content::MediaStreamDevices* audio_devices =
       request.audio_type == content::MEDIA_DEVICE_AUDIO_CAPTURE ?
           &MediaCaptureDevicesDispatcher::GetInstance()
               ->GetAudioCaptureDevices() :
           NULL;
 
   const content::MediaStreamDevices* video_devices =
       request.video_type == content::MEDIA_DEVICE_VIDEO_CAPTURE ?
           &MediaCaptureDevicesDispatcher::GetInstance()
@@ skipping 23 matching lines @@
   }
 
   if (!request.requested_video_device_id.empty() &&
       !video_devices->FindById(request.requested_video_device_id)) {
     return false;
   }
 
   return true;
 }
 
-base::string16 GetMediaPermissionsFromNavigationEntry(
-    content::NavigationEntry* navigation_entry,
-    const content::MediaStreamRequest& request) {
-  const std::string key(kMediaPermissionKeyPrefix +
-                        request.security_origin.spec());
-
-  base::string16 permissions;
-  if (!navigation_entry->GetExtraData(key, &permissions)) {
-    DCHECK(permissions.empty());
-  }
-
-  return permissions;
-}
-
-void SetMediaPermissionsForNavigationEntry(
-    content::NavigationEntry* navigation_entry,
-    const content::MediaStreamRequest& request,
-    const base::string16& permissions) {
-  const std::string key(kMediaPermissionKeyPrefix +
-                        request.security_origin.spec());
-  permissions.empty() ?
-      navigation_entry->ClearExtraData(key) :
-      navigation_entry->SetExtraData(key, permissions);
-}
-
-void SetMediaPermissionsForNavigationEntry(
-    content::NavigationEntry* navigation_entry,
-    const content::MediaStreamRequest& request,
-    bool allow_audio,
-    bool allow_video) {
-  base::string16 permissions;
-  if (allow_audio)
-    permissions += kMediaPermissionAudio;
-  if (allow_video)
-    permissions += kMediaPermissionVideo;
-  SetMediaPermissionsForNavigationEntry(navigation_entry, request, permissions);
-}
-
-bool IsRequestAllowedByNavigationEntry(
-    content::NavigationEntry* navigation_entry,
-    const content::MediaStreamRequest& request) {
-  using content::MEDIA_NO_SERVICE;
-  using content::MEDIA_DEVICE_AUDIO_CAPTURE;
-  using content::MEDIA_DEVICE_VIDEO_CAPTURE;
-
-  // If we aren't being asked for at least one of these two, fail right away.
-  if (!navigation_entry ||
-      (request.audio_type != MEDIA_DEVICE_AUDIO_CAPTURE &&
-       request.video_type != MEDIA_DEVICE_VIDEO_CAPTURE)) {
-    return false;
-  }
-
-  base::string16 permissions =
-      GetMediaPermissionsFromNavigationEntry(navigation_entry, request);
-
-  bool audio_requested_and_granted =
-      request.audio_type == MEDIA_DEVICE_AUDIO_CAPTURE &&
-      permissions.find(kMediaPermissionAudio) != base::string16::npos;
-
-  bool video_requested_and_granted =
-      request.video_type == MEDIA_DEVICE_VIDEO_CAPTURE &&
-      permissions.find(kMediaPermissionVideo) != base::string16::npos;
-
-  return
-      (audio_requested_and_granted || request.audio_type == MEDIA_NO_SERVICE) &&
-      (video_requested_and_granted || request.video_type == MEDIA_NO_SERVICE);
-}
-
-
 bool IsInKioskMode() {
   if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kKioskMode))
     return true;
 
 #if defined(OS_CHROMEOS)
   const chromeos::UserManager* user_manager = chromeos::UserManager::Get();
   return user_manager && user_manager->IsLoggedInAsKioskApp();
 #else
   return false;
 #endif
@@ skipping 130 matching lines @@
     Deny(false, content::MEDIA_DEVICE_PERMISSION_DENIED);
     return true;
   }
 
   // Check if the media default setting is set to block.
   if (IsDefaultMediaAccessBlocked()) {
     Deny(false, content::MEDIA_DEVICE_PERMISSION_DENIED);
     return true;
   }
 
-  // Check if the navigation entry has previously been granted access.
-  // We do this after the IsDefaultMediaAccessBlocked check to handle the use
-  // case where the user modifies the content settings to 'deny' after having
-  // previously granted the page access and the permissions in the
-  // NavigationEntry are out of date.
-  content::NavigationEntry* navigation_entry =
-      web_contents_->GetController().GetVisibleEntry();
-  if (IsRequestAllowedByNavigationEntry(navigation_entry, request_)) {
-    Accept(false);
-    return true;
-  }
-
   // Show the infobar.
   return false;
 }
 
 bool MediaStreamDevicesController::HasAudio() const {
   return IsDeviceAudioCaptureRequestedAndAllowed();
 }
 
 bool MediaStreamDevicesController::HasVideo() const {
   return IsDeviceVideoCaptureRequestedAndAllowed();
@@ skipping 68 matching lines @@
 
         // If either or both audio and video devices were requested but not
         // specified by id, get the default devices.
         if (get_default_audio_device || get_default_video_device) {
           MediaCaptureDevicesDispatcher::GetInstance()->
               GetDefaultDevicesForProfile(profile_,
                                           get_default_audio_device,
                                           get_default_video_device,
                                           &devices);
         }
-
-        // For pages accessed via http (not https), tag this navigation entry
-        // with the granted permissions.  This avoids repeated prompts for
-        // device access.
-        if (!IsSchemeSecure()) {
-          content::NavigationEntry* navigation_entry =
-              web_contents_->GetController().GetVisibleEntry();
-          if (navigation_entry) {
-            SetMediaPermissionsForNavigationEntry(
-                navigation_entry, request_, audio_allowed, video_allowed);
-          }
-        }
         break;
       }
       case content::MEDIA_DEVICE_ACCESS: {
         // Get the default devices for the request.
         MediaCaptureDevicesDispatcher::GetInstance()->
             GetDefaultDevicesForProfile(profile_,
                                         audio_allowed,
                                         video_allowed,
                                         &devices);
         break;
@@ skipping 29 matching lines @@
              content::MEDIA_DEVICE_NO_HARDWARE : content::MEDIA_DEVICE_OK,
          ui.Pass());
 }
 
 void MediaStreamDevicesController::Deny(
     bool update_content_setting,
     content::MediaStreamRequestResult result) {
   DLOG(WARNING) << "MediaStreamDevicesController::Deny: " << result;
   NotifyUIRequestDenied();
 
-  // Clear previously allowed permissions from the navigation entry if any.
-  content::NavigationEntry* navigation_entry =
-      web_contents_->GetController().GetVisibleEntry();
-  if (navigation_entry) {
-    SetMediaPermissionsForNavigationEntry(
-        navigation_entry, request_, false, false);
-  }
-
   if (update_content_setting) {
     CHECK_EQ(content::MEDIA_DEVICE_PERMISSION_DENIED, result);
     SetPermission(false);
   }
 
   content::MediaResponseCallback cb = callback_;
   callback_.Reset();
   cb.Run(content::MediaStreamDevices(),
          result,
          scoped_ptr<content::MediaStreamUI>());
@@ skipping 273 matching lines @@
           it->second.permission == MEDIA_ALLOWED);
 }
 
 bool MediaStreamDevicesController::IsDeviceVideoCaptureRequestedAndAllowed()
     const {
   MediaStreamTypeSettingsMap::const_iterator it =
       request_permissions_.find(content::MEDIA_DEVICE_VIDEO_CAPTURE);
   return (it != request_permissions_.end() &&
           it->second.permission == MEDIA_ALLOWED);
 }