New issue
Advanced search Search tips

Issue 871914 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Mash: Support UserActivityController

Project Member Reported by steve...@chromium.org, Aug 7

Issue description

UserActivityController::UserActivityController() {
  // TODO(jiameng): video detector below doesn't work with MASH. Temporary
  // solution is to disable logging if we're under MASH env.
  if (chromeos::GetDeviceType() != chromeos::DeviceType::kChromebook ||
      !features::IsAshInBrowserProcess())
    return;

There are no direct header dependencies here, but we will need to enable UserActivityController for Mash at some point.

 
Summary: Mash: Support UserActivityController (was: <ash: Support UserActivityController)
Cc: napper@chromium.org
Thanks for the update. Would you please let us know when mash will be rolled out? We currently don't have resources to work on this, but we'll make sure it's done before mash is rolled out.
Thanks.
Labels: Proj-Mustash
Cc: sky@chromium.org
Labels: -Proj-Mustash Proj-Mash-SingleProcess
sky, do you know if the video detector code here will work in SingleProcessMash?

https://cs.chromium.org/chromium/src/chrome/browser/chromeos/power/ml/user_activity_controller.cc?q=useractivitycontroller&sq=package:chromium&l=21

go/singleprocessmash
I believe if that code is changed to use Shell's env it will work in single-process-mash.
Owner: ----
Status: Available (was: Assigned)
Labels: -Proj-Mash-SingleProcess Proj-Mash-MultiProcess
looks as though single process was fixed with 913f1641f753a2a10c0dfc3b so updating labels

Sign in to add a comment