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

Issue 699589 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: 3
NextAction: 2017-03-10
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

Add sound volume to device attribute reporting

Project Member Reported by isandrk@chromium.org, Mar 8 2017

Issue description

It's useful for admins to remotely check the volume level.

Design doc: go/remote-audio-management

More info at crbug.com/682792
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 10 2017

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

commit f4366520e5828318c92113012e02f1027779a792
Author: isandrk <isandrk@chromium.org>
Date: Fri Mar 10 11:03:09 2017

Add sound volume to device attribute reporting

This CL adds another device attribute to the report which is uploaded to admin console.

More context can be found by reading the design doc go/remote-audio-management

BUG= 699589 

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

[modify] https://crrev.com/f4366520e5828318c92113012e02f1027779a792/chrome/browser/chromeos/policy/device_status_collector.cc
[modify] https://crrev.com/f4366520e5828318c92113012e02f1027779a792/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
[modify] https://crrev.com/f4366520e5828318c92113012e02f1027779a792/components/policy/proto/device_management_backend.proto

Labels: Merge-Request-58
Status: Fixed (was: Started)
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 11 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), bhthompson@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 13 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bda690ff633374ef6d0cbfd460c6475c29093516

commit bda690ff633374ef6d0cbfd460c6475c29093516
Author: Ivan Sandrk <isandrk@google.com>
Date: Mon Mar 13 10:35:05 2017

Merge of "Add sound volume to device attribute reporting" to M58

This CL adds another device attribute to the report which is uploaded to admin console.

More context can be found by reading the design doc go/remote-audio-management

BUG= 699589 

Review-Url: https://codereview.chromium.org/2736903004
Cr-Commit-Position: refs/heads/master@{#456039}
(cherry picked from commit f4366520e5828318c92113012e02f1027779a792)

Review-Url: https://codereview.chromium.org/2745073002 .
Cr-Commit-Position: refs/branch-heads/3029@{#143}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/bda690ff633374ef6d0cbfd460c6475c29093516/chrome/browser/chromeos/policy/device_status_collector.cc
[modify] https://crrev.com/bda690ff633374ef6d0cbfd460c6475c29093516/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
[modify] https://crrev.com/bda690ff633374ef6d0cbfd460c6475c29093516/components/policy/proto/device_management_backend.proto

Can be tested on the client after this feature is implemented in the CPanel.

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment