New issue
Advanced search Search tips

Issue 715449 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Verify camera minimum resolution

Project Member Reported by wuchengli@chromium.org, Apr 26 2017

Issue description

ChromeOS spec requires back camera to be at least 3MP and front camera to be at least 720p. Let's check it in both arc-camera.eclass and camera_V4L2.

Several devices do not have 3MP back camera. Let me check with PM first.
 
I confirmed the back camera should be at least 1920x1080. Please add the test case in arc-camera.eclass and camera_HAL3. This test case shouldn't be added in camera_V4L2 because camera_V4L2 should test V4L2 level. Camera facing is not reported by V4L2.
Cc: henryhsu@chromium.org
 Issue 726282  has been merged into this issue.
Copy from 726282:
It was decided the minimum resolution of back camera on ChromeOS should be 1920x1080 or above. Let's add a test case in camera_V4L2 for that. We only need to test this when the pid/vid exists in camera_characteristics.conf.

Now camera_V4L2 can read camera config. I'll add a test in camera_V4L2
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/95c0580459a9b60d9f6924094e553b7af868c6dd

commit 95c0580459a9b60d9f6924094e553b7af868c6dd
Author: Heng-Ruey Hsu <henryhsu@chromium.org>
Date: Fri Jun 23 10:14:50 2017

camera_V4L2: Add minimum resolution check test case

For world-facing camera, the maximum resolution should be at least
1080p. For user-facing camera, the maximum resolution should be at least
720p. This test case only for devices which have camera config file.

BUG= chromium:715449 
TEST=test_that -b ${BOARD} ${IP} camera_V4L2

Change-Id: I4285515029e437b239a942aec7a9ecc1730c7ba6
Reviewed-on: https://chromium-review.googlesource.com/523204
Commit-Ready: Heng-ruey Hsu <henryhsu@google.com>
Tested-by: Heng-ruey Hsu <henryhsu@google.com>
Reviewed-by: Heng-ruey Hsu <henryhsu@google.com>

[modify] https://crrev.com/95c0580459a9b60d9f6924094e553b7af868c6dd/client/site_tests/camera_V4L2/src/media_v4l2_test.cc
[modify] https://crrev.com/95c0580459a9b60d9f6924094e553b7af868c6dd/client/site_tests/camera_V4L2/src/common_types.h

Status: Fixed (was: Assigned)

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

Status: Archived (was: Fixed)

Sign in to add a comment