New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 783876 link

Starred by 1 user
Status: Fixed
Owner:
xhw...@chromium.org
Closed: Jan 2018
Cc:
█ roc...@chromium.org
jrumm...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Use instance ID instead of user ID for different CDM types

Project Member Reported by xhw...@chromium.org, Nov 10 2017 
See  issue 664364  for previous discussion, where instance ID should be the right option but it didn't work, so that we had to use user ID. Now according to rockot@, instance ID should work. This issue tracks the work to switch to use instance ID.
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 14 2017 

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

commit 74f912b0198e74ae2144b70531bd7528c609f4f5
Author: Xiaohan Wang <xhwang@chromium.org>
Date: Tue Nov 14 20:33:58 2017

media: Use instance ID to run different CDM types in different processes

Instance ID is the right way to group service processes. Previously we
use user ID because there's an issue using instance ID. Now that issue
is fixed, this CL switch back to use instance ID.

BUG= 783876 
TEST=Mojo/ECKEncryptedMediaTest.MultipleCdmTypes* browser_tests pass

Change-Id: Ia15e1a94579e92a53156292b5e267dc67d19a68b
Reviewed-on: https://chromium-review.googlesource.com/765106
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516408}
[modify] https://crrev.com/74f912b0198e74ae2144b70531bd7528c609f4f5/content/browser/media/media_interface_proxy.cc

Comment 2 by xhw...@chromium.org, Jan 3 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment