Rotate frames correctly for back camera

media/capture/video/linux/video_capture_device_chromeos.h

 // Copyright 2014 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.
 
 #ifndef MEDIA_CAPTURE_VIDEO_LINUX_VIDEO_CAPTURE_DEVICE_CHROMEOS_H_
 #define MEDIA_CAPTURE_VIDEO_LINUX_VIDEO_CAPTURE_DEVICE_CHROMEOS_H_
 
+#include "base/lazy_instance.h"
 #include "base/macros.h"
+#include "media/capture/video/linux/camera_characteristics.h"
 #include "media/capture/video/linux/video_capture_device_linux.h"
 
 namespace display {
 class Display;
 }  // namespace display
 
 namespace media {
 
 // This class is functionally the same as VideoCaptureDeviceLinux, with the
 // exception that it is aware of the orientation of the internal Display.  When
 // the internal Display is rotated, the frames captured are rotated to match.
 class VideoCaptureDeviceChromeOS : public VideoCaptureDeviceLinux {
  public:
   explicit VideoCaptureDeviceChromeOS(
       scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner,
       const VideoCaptureDeviceDescriptor& device_descriptor);
   ~VideoCaptureDeviceChromeOS() override;
 
+ protected:
+  void SetRotation(int rotation) override;
+
  private:
   class ScreenObserverDelegate;
 
   void SetDisplayRotation(const display::Display& display);
   scoped_refptr<ScreenObserverDelegate> screen_observer_delegate_;
+  int lens_facing_;
+  base::LazyInstance<CameraCharacteristics> camera_characteristics_;

[kcwu, 2016/11/24 15:48:52]

This should be static or global, otherwise no point to use LazyInstance<>.

[shenghao, 2016/11/28 08:52:04]

Done.

   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoCaptureDeviceChromeOS);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_CAPTURE_VIDEO_LINUX_VIDEO_CAPTURE_DEVICE_CHROMEOS_H_