Make TabCapture requests use the target render process and render view id's for UI 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/stringprintf.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/extensions/api/tab_capture/tab_capture_registry.h"
 #include "chrome/browser/extensions/api/tab_capture/tab_capture_registry_factory.h"
 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
 #include "chrome/browser/media/media_internals.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 32 matching lines @@
       has_audio_(content::IsAudioMediaType(request->audio_type)),
       has_video_(content::IsVideoMediaType(request->video_type)) {
   DCHECK(request);
 }
 
 MediaStreamDevicesController::~MediaStreamDevicesController() {}
 
 bool MediaStreamDevicesController::DismissInfoBarAndTakeActionOnSettings() {
   if (request_.audio_type == content::MEDIA_TAB_AUDIO_CAPTURE ||
       request_.video_type == content::MEDIA_TAB_VIDEO_CAPTURE) {
-    Accept(false);
+    extensions::TabCaptureRegistry* registry =
+        extensions::TabCaptureRegistryFactory::GetForProfile(profile_);
+
+    std::string device_id = base::StringPrintf(
+        "%i:%i", request_.render_process_id, request_.render_view_id);

[no longer working on chromium, 2012/12/05 17:38:37]

this code looks a bit odd, can you consider changeing the type to 
typedef std::map<std::pair<int, int>,  TabCaptureRequest>
DeviceCaptureRequestMap;

+
+    if (!registry->VerifyRequest(device_id)) {
+      Deny();
+    } else {
+      content::MediaStreamDevices devices;
+
+      if (request_.video_type == content::MEDIA_TAB_VIDEO_CAPTURE) {
+        devices.push_back(content::MediaStreamDevice(
+            content::MEDIA_TAB_VIDEO_CAPTURE, "", ""));
+      }
+      if (request_.audio_type == content::MEDIA_TAB_AUDIO_CAPTURE) {
+        devices.push_back(content::MediaStreamDevice(
+            content::MEDIA_TAB_AUDIO_CAPTURE, "", ""));
+      }
+
+      callback_.Run(devices);
+    }
+
+    return true;
   }
 
   // Deny the request if the security origin is empty, this happens with
   // file access without |--allow-file-access-from-files| flag.
   if (request_.security_origin.is_empty()) {
     Deny();
     return true;
   }
 
   // Deny the request if there is no device attached to the OS.
@@ skipping 127 matching lines @@
 
   ContentSettingsPattern primary_pattern =
       ContentSettingsPattern::FromURLNoWildcard(request_.security_origin);
   profile_->GetHostContentSettingsMap()->SetWebsiteSetting(
       primary_pattern,
       ContentSettingsPattern::Wildcard(),
       CONTENT_SETTINGS_TYPE_MEDIASTREAM,
       NO_RESOURCE_IDENTIFIER,
       dictionary_value);
 }