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

Issue 808144 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Add audio-visual perception to media perception private api

Project Member Reported by weigua@chromium.org, Feb 1 2018

Issue description

MRI team is adding human presence detection feature into media perception backend. The perception results will be piped to javascript frontend through media perception private api.
 
Status: Assigned (was: Untriaged)
[Extensions Triage]: Marking as assigned.

Comment 3 by weigua@chromium.org, Feb 12 2018

Labels: Merge-Request-65
Project Member

Comment 4 by sheriffbot@chromium.org, Feb 13 2018

Labels: -Merge-Request-65 Merge-Reject-65 Hotlist-Merge-Reject
The bug is marked as P3 or Feature. It should not be merged as M65 is in beta. 
Please contact the approriate milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

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

Comment 5 by weigua@chromium.org, Feb 13 2018

Labels: -Type-Feature -Hotlist-Merge-Reject -Merge-Reject-65 Merge-Request-65 Type-Bug
PTAL. We need to merge this back to M64.

Comment 6 by weigua@chromium.org, Feb 13 2018

Sorry I mean M65.
Project Member

Comment 7 by sheriffbot@chromium.org, Feb 13 2018

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

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Review -Merge-Review-65 Merge-Approved-65
If this is really not a new feature, and/or this is specific to CFM and has no risk of destabilizing the rest of the fleet (including Windows/Mac/Linux) then we can merge this back.

By merging this you affirm that to the best of your knowledge that this is limited in scope as described above.

Comment 9 by weigua@chromium.org, Feb 20 2018

Thanks. This private API will only be used by Hangouts App on CfM. It is to unlock the new occupancy detection feature.
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 21 2018

Labels: -merge-approved-65 merge-merged-3325
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bad5b99ec6308ba779f555a2f0b25add918670c4

commit bad5b99ec6308ba779f555a2f0b25add918670c4
Author: Wei Guan <weigua@chromium.org>
Date: Wed Feb 21 16:10:59 2018

Add AudioVisualPerception to MediaPerceptionPrivate API.

Bug: 808144
Change-Id: I04a25f1776b7299dfc9273108e5fc38ed8c34290
Reviewed-on: https://chromium-review.googlesource.com/897895
Commit-Queue: Wei Guan <weigua@chromium.org>
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#536140}(cherry picked from commit 9932813b326c3d4b70e2b0ca645b12e5890cf74f)
Reviewed-on: https://chromium-review.googlesource.com/927022
Reviewed-by: Simon Que <sque@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#531}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/bad5b99ec6308ba779f555a2f0b25add918670c4/chromeos/dbus/proto/media_perception.proto
[modify] https://crrev.com/bad5b99ec6308ba779f555a2f0b25add918670c4/extensions/browser/api/media_perception_private/conversion_utils.cc
[modify] https://crrev.com/bad5b99ec6308ba779f555a2f0b25add918670c4/extensions/browser/api/media_perception_private/conversion_utils_unittest.cc
[modify] https://crrev.com/bad5b99ec6308ba779f555a2f0b25add918670c4/extensions/common/api/media_perception_private.idl

Sign in to add a comment