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

Issue 810519 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

Chameleon: Split the multimedia test extension into two: display and audio

Project Member Reported by waihong@chromium.org, Feb 8 2018

Issue description

After the recent change of the Chrome display APIs, using them requires the autotestPrivate permission which can only be applied to a extension. But the audio APIs can only be used in an app, not an extension. We have to split the current single extension/app into two: one as an extension for display APIs and one as an app for audio APIs.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/018730814bafdd4647a8370721960cb1f3d2c6d3

commit 018730814bafdd4647a8370721960cb1f3d2c6d3
Author: Wai-Hong Tam <waihong@google.com>
Date: Thu Feb 15 04:39:03 2018

Chameleon: Split extension into two for audio and display APIs

Chrome display APIs require the autotestPrivate permission which can
only be applied to an extension. But the audio APIs can only be used
in an app, not an extension. We need to split the current single
extension/app into two: one as an extension for display APIs and
one as an app for audio APIs.

The display test extension inherits the original multimedia test
extension (whitelisted for autotestPrivate) and the audio test
extension is new.

BUG= chromium:810519 
TEST=Ran suite:chameleon_hdmi_perbuild and suite:chameleon_audio_perbuild

Change-Id: I7f9660ad25121f208aed7fa7b40882f90076382f
Reviewed-on: https://chromium-review.googlesource.com/917008
Commit-Ready: Wai-Hong Tam <waihong@google.com>
Tested-by: Wai-Hong Tam <waihong@google.com>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>

[rename] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/display_test_extension/manifest.json
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/constants.py
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/site_tests/display_ClientChameleonConnection/display_ClientChameleonConnection.py
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/server/site_tests/audiovideo_AVSync/audiovideo_AVSync.py
[add] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/audio_test_extension/manifest.json
[add] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/audio_test_extension/background.js
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/audio_facade_native.py
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/facade_resource.py
[rename] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/display_test_extension/background.js
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/cros/multimedia/display_facade_native.py
[modify] https://crrev.com/018730814bafdd4647a8370721960cb1f3d2c6d3/client/site_tests/video_GlitchDetection/video_GlitchDetection.py

Status: Fixed (was: Available)

Sign in to add a comment