New issue
Advanced search Search tips

Issue 607112 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

MediaStream IPC uses GURL instead of url::Origin for security origins

Project Member Reported by guidou@chromium.org, Apr 27 2016

Issue description

url::Origin is the preferred way to represent security origins in IPC, so we should update media_stream_messages.h and related code to reflect this.
 

Comment 1 by guidou@chromium.org, Apr 27 2016

Components: Internals>Media Blink>WebRTC
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 30 2016

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

commit c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0
Author: guidou <guidou@chromium.org>
Date: Sat Apr 30 18:00:37 2016

Use url::Origin to represent security origins in MediaStream-related code.

BUG= 607112 

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

[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/chrome/browser/extensions/api/webrtc_audio_private/webrtc_audio_private_api.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/chrome/browser/extensions/api/webrtc_audio_private/webrtc_audio_private_api.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/chrome/browser/extensions/api/webrtc_audio_private/webrtc_audio_private_browsertest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/audio_renderer_host.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/audio_renderer_host.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_dispatcher_host.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_dispatcher_host.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_dispatcher_host_unittest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_manager.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_manager.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_manager_unittest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_ui_proxy.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_ui_proxy.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/media_stream_ui_proxy_unittest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/renderer_host/media/video_capture_host_unittest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/browser/speech/speech_recognition_manager_impl.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/common/media/media_stream_messages.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/public/browser/media_device_id.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/public/browser/media_device_id.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/media_stream_dispatcher.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/media_stream_dispatcher.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/media_stream_dispatcher_unittest.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/mock_media_stream_dispatcher.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/mock_media_stream_dispatcher.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/media/user_media_client_impl.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/content/renderer/pepper/pepper_media_device_manager.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/extensions/browser/api/webcam_private/webcam_private_api_chromeos.cc
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/third_party/WebKit/Source/modules/mediastream/MediaStreamTrackSourcesRequestImpl.cpp
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/third_party/WebKit/Source/modules/mediastream/MediaStreamTrackSourcesRequestImpl.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/third_party/WebKit/Source/platform/exported/WebMediaStreamTrackSourcesRequest.cpp
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/third_party/WebKit/Source/platform/mediastream/MediaStreamTrackSourcesRequest.h
[modify] https://crrev.com/c6dd4461f54c7d3f4e87bc6626e4a81f4b36dba0/third_party/WebKit/public/platform/WebMediaStreamTrackSourcesRequest.h

Status: Fixed (was: Started)

Sign in to add a comment