New issue
Advanced search Search tips

Issue 829233 link

Starred by 1 user
Status: Assigned
Owner:
xhw...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Enable MediaServiceTest.InitializeCdm* tests on Android

Project Member Reported by xhw...@chromium.org, Apr 5 2018 
MediaServiceTest runs the service out-of-process as a standalone service. This seems to be causing trouble on Android when we push test dependencies (media.service) over to the test device. I'll need to take a deeper look to really understand why it's failing.

An alternative solution would be to run the test in-process as packaged service. Given the fact that we always run MediaService as packaged service in production, this makes sense to me.
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 6 2018 

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

commit bb003029206384246959ad15a1e85e529c2bbafa
Author: Xiaohan Wang <xhwang@chromium.org>
Date: Fri Apr 06 05:07:33 2018

media: Various fixes on MediaServiceTest

- Remove InitializeRenderer_InvalidConfig, which causes
  deserialization error and test timeout.
- Reenable tests now that  http://crbug.com/594977  is fixed.
- Disable InitializeCdm* tests on Android. See Bug.

Bug: 829233
Test: Various test fixes.
Change-Id: Iaa4ac72a8e60a88559fa198e9f3c1da16aa0beb4
Reviewed-on: https://chromium-review.googlesource.com/996959
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548678}
[modify] https://crrev.com/bb003029206384246959ad15a1e85e529c2bbafa/media/mojo/services/media_service_unittest.cc

Comment 2 by xhw...@chromium.org, Apr 13 2018

Labels: -M-68

Comment 3 by xhw...@chromium.org, Oct 9

Summary: Enable MediaServiceTest.InitializeCdm* tests on Android (was: MediaServiceTest.InitializeCdm* fails on Android)
►

Sign in to add a comment