Rotate frames correctly for back camera

media/capture/video/linux/video_capture_device_linux.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/capture/video/linux/video_capture_device_linux.h"
 
 #include <stddef.h>
 
 #include <list>
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "build/build_config.h"
+#include "media/capture/video/linux/camera_characteristics.h"
 #include "media/capture/video/linux/v4l2_capture_delegate.h"
 
 #if defined(OS_OPENBSD)
 #include <sys/videoio.h>
 #else
 #include <linux/videodev2.h>
 #endif
 
 namespace media {
 
 // Translates Video4Linux pixel formats to Chromium pixel formats.
 // static
 VideoPixelFormat VideoCaptureDeviceLinux::V4l2FourCcToChromiumPixelFormat(
     uint32_t v4l2_fourcc) {
   return V4L2CaptureDelegate::V4l2FourCcToChromiumPixelFormat(v4l2_fourcc);
 }
 
 // Gets a list of usable Four CC formats prioritized.
 // static
 std::list<uint32_t> VideoCaptureDeviceLinux::GetListOfUsableFourCCs(
     bool favour_mjpeg) {
   return V4L2CaptureDelegate::GetListOfUsableFourCcs(favour_mjpeg);
 }
 
 VideoCaptureDeviceLinux::VideoCaptureDeviceLinux(
     const VideoCaptureDeviceDescriptor& device_descriptor)
-    : v4l2_thread_("V4L2CaptureThread"),
-      device_descriptor_(device_descriptor) {}
+    : v4l2_thread_("V4L2CaptureThread"), device_descriptor_(device_descriptor) {
+  CameraCharacteristics characteristics;

[wuchengli, 2016/11/22 07:59:34]

base::LazyInstance<CameraCharacteristics> g_camera_characteristics_ = 
    LAZY_INSTANCE_INITIALIZER;



is_back_camera_ =
g_camera_characteristics_.Get().GetCameraFacing(device_descriptor.model_id);

[shenghao, 2016/11/24 07:13:04]

Done.

+  is_back_camera_ =
+      characteristics.GetCameraFacing(device_descriptor.model_id) ==
+      DeviceInfo::LENS_FACING_BACK;
+}
 
 VideoCaptureDeviceLinux::~VideoCaptureDeviceLinux() {
   // Check if the thread is running.
   // This means that the device has not been StopAndDeAllocate()d properly.
   DCHECK(!v4l2_thread_.IsRunning());
   v4l2_thread_.Stop();
 }
 
 void VideoCaptureDeviceLinux::AllocateAndStart(
     const VideoCaptureParams& params,
@@ skipping 54 matching lines @@
   if (!v4l2_thread_.IsRunning())
     return;  // Wrong state.
   v4l2_thread_.task_runner()->PostTask(
       FROM_HERE,
       base::Bind(&V4L2CaptureDelegate::SetPhotoOptions, capture_impl_,
                  base::Passed(&settings), base::Passed(&callback)));
 }
 
 void VideoCaptureDeviceLinux::SetRotation(int rotation) {
   if (v4l2_thread_.IsRunning()) {
+    // We assume external camera is facing the users. If not, the users can
+    // rotate the camera manually by theirselves.

[wuchengli, 2016/11/22 03:47:06]

s/theirselves/themselves/

[shenghao, 2016/11/24 07:13:04]

Done.

+    if (is_back_camera_) {
+      // Original frame when screen_rotation = 0
+      // -----------------------
+      // |                     |
+      // |          *          |
+      // |         * *         |
+      // |        *****        |

[wuchengli, 2016/11/22 03:47:06]

Great diagram! Can you make the A of the first one bigger to match the sizes of
the other two?

[shenghao, 2016/11/24 07:13:04]

Done.

+      // |       *     *       |
+      // |                     |
+      // -----------------------
+      //
+      // screen_rotation = 90, this is what back camera sees
+      // -----------------------
+      // |    ********         |
+      // |       *   ****      |
+      // |       *      ***    |
+      // |       *      ***    |
+      // |       *   ****      |
+      // |    ********         |
+      // -----------------------
+      //
+      // screen_rotation = 90, this is what front camera sees
+      // -----------------------
+      // |         ********    |
+      // |      ****   *       |
+      // |    ***      *       |
+      // |    ***      *       |
+      // |      ****   *       |
+      // |         ********    |
+      // -----------------------
+      //
+      // Therefore, for back camera, we need to rotate (360 - screen_rotation).
+      rotation = (360 - rotation) % 360;
+    }
     v4l2_thread_.task_runner()->PostTask(
         FROM_HERE,
         base::Bind(&V4L2CaptureDelegate::SetRotation, capture_impl_, rotation));
   }
 }
 
 // static
 int VideoCaptureDeviceLinux::TranslatePowerLineFrequencyToV4L2(
     PowerLineFrequency frequency) {
   switch (frequency) {
     case media::PowerLineFrequency::FREQUENCY_50HZ:
       return V4L2_CID_POWER_LINE_FREQUENCY_50HZ;
     case media::PowerLineFrequency::FREQUENCY_60HZ:
       return V4L2_CID_POWER_LINE_FREQUENCY_60HZ;
     default:
       // If we have no idea of the frequency, at least try and set it to AUTO.
       return V4L2_CID_POWER_LINE_FREQUENCY_AUTO;
   }
 }
 
 }  // namespace media