camera: Collect metrics for camera facing per session
Originally we collect metrics for camera facing in GetCameraInfo(). But
we should move it to OpenDevice() so that we can collect it per session.
BUG=b:175167869
TEST=None
Change-Id: I405dd7344dc3a8560f42b19e21818ea74af23828
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2592185
Tested-by: Wei Lee <wtlee@chromium.org>
Auto-Submit: Wei Lee <wtlee@chromium.org>
Reviewed-by: Shik Chen <shik@chromium.org>
Commit-Queue: Wei Lee <wtlee@chromium.org>
diff --git a/camera/hal_adapter/camera_hal_adapter.cc b/camera/hal_adapter/camera_hal_adapter.cc
index 20a8910..5168724 100644
--- a/camera/hal_adapter/camera_hal_adapter.cc
+++ b/camera/hal_adapter/camera_hal_adapter.cc
@@ -187,6 +187,7 @@
return -ENODEV;
}
device_adapters_.at(camera_id)->Bind(std::move(device_ops_request));
+ camera_metrics_->SendCameraFacing(info.facing);
camera_metrics_->SendOpenDeviceLatency(
session_timer_map_[camera_id].Elapsed());
@@ -224,8 +225,6 @@
return ret;
}
- camera_metrics_->SendCameraFacing(info.facing);
-
LOGF(INFO) << "camera_id = " << camera_id << ", facing = " << info.facing;
if (VLOG_IS_ON(2)) {
