Add audio-visual perception to media perception private api |
|||||||
Issue descriptionMRI 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.
,
Feb 12 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9932813b326c3d4b70e2b0ca645b12e5890cf74f commit 9932813b326c3d4b70e2b0ca645b12e5890cf74f Author: Wei Guan <weigua@chromium.org> Date: Mon Feb 12 18:45:57 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-Commit-Position: refs/heads/master@{#536140} [modify] https://crrev.com/9932813b326c3d4b70e2b0ca645b12e5890cf74f/chromeos/dbus/proto/media_perception.proto [modify] https://crrev.com/9932813b326c3d4b70e2b0ca645b12e5890cf74f/extensions/browser/api/media_perception_private/conversion_utils.cc [modify] https://crrev.com/9932813b326c3d4b70e2b0ca645b12e5890cf74f/extensions/browser/api/media_perception_private/conversion_utils_unittest.cc [modify] https://crrev.com/9932813b326c3d4b70e2b0ca645b12e5890cf74f/extensions/common/api/media_perception_private.idl
,
Feb 12 2018
,
Feb 13 2018
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
,
Feb 13 2018
PTAL. We need to merge this back to M64.
,
Feb 13 2018
Sorry I mean M65.
,
Feb 13 2018
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
,
Feb 20 2018
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.
,
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.
,
Feb 21 2018
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 |
|||||||
Comment 1 by karandeepb@chromium.org
, Feb 9 2018