Move VideoFrame and related declarations to webrtc/api/video.

webrtc/test/fake_texture_frame.h

 /*
  *  Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 #ifndef WEBRTC_TEST_FAKE_TEXTURE_FRAME_H_
 #define WEBRTC_TEST_FAKE_TEXTURE_FRAME_H_
 
+#include "webrtc/api/video/video_frame.h"
 #include "webrtc/base/checks.h"
 #include "webrtc/common_video/include/video_frame_buffer.h"
-#include "webrtc/video_frame.h"
 
 namespace webrtc {
 namespace test {
 
 class FakeNativeHandle {
  public:
   static VideoFrame CreateFrame(FakeNativeHandle* native_handle,
                                 int width,
                                 int height,
                                 uint32_t timestamp,
                                 int64_t render_time_ms,
                                 VideoRotation rotation);
 };
 
 class FakeNativeHandleBuffer : public NativeHandleBuffer {
  public:
   FakeNativeHandleBuffer(void* native_handle, int width, int height)
       : NativeHandleBuffer(native_handle, width, height) {}
 
   ~FakeNativeHandleBuffer() {
     delete reinterpret_cast<FakeNativeHandle*>(native_handle_);
   }
 
  private:
   rtc::scoped_refptr<VideoFrameBuffer> NativeToI420Buffer() override {
     rtc::scoped_refptr<I420Buffer> buffer = I420Buffer::Create(width_, height_);
-    int half_height = (height_ + 1) / 2;
-    int half_width = (width_ + 1) / 2;
-    memset(buffer->MutableDataY(), 0, height_ * width_);
-    memset(buffer->MutableDataU(), 0, half_height * half_width);
-    memset(buffer->MutableDataV(), 0, half_height * half_width);
+    buffer->InitializeData();

[the sun, 2016/12/05 22:36:29]

SetToBlack()? Do we really need both SetToBlack and InitializeData?

[nisse-webrtc, 2016/12/06 12:02:15]

Probably not. Do you want InitializeData deleted in this cl?

We also need to decide what to do about SetToBlack, if it should be converted
from a regular method to a static method or plain function.

[the sun, 2016/12/07 15:43:51]

Not necessarily, but if you intend to delete it, don't use it here.
If you keep Rotate() etc as static functions in the class, then make it one too.

[nisse-webrtc, 2016/12/13 15:08:02]

Done. Have to keep the old method for compatibility with Chrome, so I named the
new static version SetBlack. Ok?

[the sun, 2016/12/13 15:29:28]

Sure.

     return buffer;
   }
 };
 
 }  // namespace test
 }  // namespace webrtc
 #endif  //  WEBRTC_TEST_FAKE_TEXTURE_FRAME_H_