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

Issue 755399 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Task



Sign in to add a comment

[CFM Test Automation] Verify the hotrod app sets available speaker as preferred when preferred speakers are not available

Project Member Reported by dsunk...@chromium.org, Aug 14 2017

Issue description

[CFM Test Automation] Verify the hotrod app sets available speaker as preferred when preferred speakers are not available

Automated test(enterprise_CFM_USBPeripheralHotplugStress) fail when there is a speaker (HDMI speaker = built in monitor) as it detects monitor's speaker and sets as preferred speaker (which is intended behavior). 

The script needs to be updated to only check for whitelist speaker set as preferred when it is plugged in.

When the whitelist speaker is not plugged in, available speaker is updated as preferred speaker in cfm.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 17 2017

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

commit 4fa6ca6b05ec64a278cf18b9931374b7e91796dc
Author: Dinesh Kumar Sunkara <dsunkara@chromium.org>
Date: Thu Aug 17 18:59:45 2017

[AutoTest] Updated enterprise_CFM_USBPeripheralHotplugStress

Updated enterprise_CFM_USBPeripheralHotplugStress to check if
available speaker is set as preferred speaker when preferred
speaker are not available. Also, added a check to only check
for whitelist speaker set as preferred when it is plugged in.

BUG= chromium:755399 
TEST=Tested locally on Guado with external speakers
pre-cq-configs: bluestreak-pre-cq

Change-Id: I2a38b5470d24a2e2d81a2615a28b84970468fcb3
Reviewed-on: https://chromium-review.googlesource.com/614918
Trybot-Ready: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Commit-Queue: Harpreet Grewal <harpreet@chromium.org>

[modify] https://crrev.com/4fa6ca6b05ec64a278cf18b9931374b7e91796dc/server/site_tests/enterprise_CFM_USBPeripheralHotplugStress/enterprise_CFM_USBPeripheralHotplugStress.py

Labels: -Pri-3 M-61 Merge-Request-61 M-62 Pri-2
Status: Started (was: Untriaged)
Project Member

Comment 3 by sheriffbot@chromium.org, Aug 21 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: M61 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Started)
Labels: -Merge-Review-61 Merge-Approved-61
Approving merge to M61 Chrome OS.
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 22 2017

Labels: merge-merged-release-R61-9765.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/0bd35b858597b1a0a8b39cc88db135876fe99c27

commit 0bd35b858597b1a0a8b39cc88db135876fe99c27
Author: Dinesh Kumar Sunkara <dsunkara@chromium.org>
Date: Tue Aug 22 00:33:08 2017

[AutoTest] Updated enterprise_CFM_USBPeripheralHotplugStress

Updated enterprise_CFM_USBPeripheralHotplugStress to check if
available speaker is set as preferred speaker when preferred
speaker are not available. Also, added a check to only check
for whitelist speaker set as preferred when it is plugged in.

BUG= chromium:755399 
TEST=Tested locally on Guado with external speakers
pre-cq-configs: bluestreak-pre-cq

Change-Id: I2a38b5470d24a2e2d81a2615a28b84970468fcb3
Reviewed-on: https://chromium-review.googlesource.com/614918
Trybot-Ready: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Tested-by: Harpreet Grewal <harpreet@chromium.org>
Commit-Queue: Harpreet Grewal <harpreet@chromium.org>
(cherry picked from commit 4fa6ca6b05ec64a278cf18b9931374b7e91796dc)
Reviewed-on: https://chromium-review.googlesource.com/624871
Tested-by: Dinesh Kumar Sunkara <dsunkara@google.com>

[modify] https://crrev.com/0bd35b858597b1a0a8b39cc88db135876fe99c27/server/site_tests/enterprise_CFM_USBPeripheralHotplugStress/enterprise_CFM_USBPeripheralHotplugStress.py

Project Member

Comment 7 by sheriffbot@chromium.org, Aug 25 2017

Cc: keta...@chromium.org
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-61

Sign in to add a comment