New issue
Advanced search Search tips

Issue 720025 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 657733



Sign in to add a comment

Do not pass security origin to MediaDevicesDispatcherHost

Project Member Reported by guidou@chromium.org, May 9 2017

Issue description

It should get the origin from the corresponding RenderFrameHost.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a72075abe9fe56104e33384f5876daa42a946a42

commit a72075abe9fe56104e33384f5876daa42a946a42
Author: guidou <guidou@chromium.org>
Date: Mon May 15 08:36:17 2017

Do not pass the origin to MediaDevicesDispatcherHost.

Instead get the origin from the corresponding RenderFrameHost.
Unique origins are now treated like any other insecure origin.

BUG= 720025 

Review-Url: https://codereview.chromium.org/2872913003
Cr-Commit-Position: refs/heads/master@{#471699}

[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/bad_message.h
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/media/media_devices_permission_checker.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/media/media_devices_permission_checker.h
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/renderer_host/media/audio_output_authorization_handler.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/renderer_host/media/media_devices_dispatcher_host.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/renderer_host/media/media_devices_dispatcher_host.h
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/browser/renderer_host/media/media_devices_dispatcher_host_unittest.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/common/media/media_devices.mojom
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/media/media_devices_event_dispatcher.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/media/media_devices_event_dispatcher.h
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/media/media_devices_event_dispatcher_unittest.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/media/user_media_client_impl.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/media/user_media_client_impl_unittest.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/pepper/pepper_device_enumeration_host_helper.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/pepper/pepper_device_enumeration_host_helper.h
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/pepper/pepper_device_enumeration_host_helper_unittest.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/pepper/pepper_media_device_manager.cc
[modify] https://crrev.com/a72075abe9fe56104e33384f5876daa42a946a42/content/renderer/pepper/pepper_media_device_manager.h

Comment 2 by guidou@chromium.org, May 15 2017

Status: Fixed (was: Started)

Sign in to add a comment