RTCVideoDecoder: Using I420VideoFrame directly instead of to-be-removed TextureVideoFrame (previous…

RTCVideoDecoder: Using I420VideoFrame directly instead of to-be-removed TextureVideoFrame (previously this typeddefed to I420VideoFrame).

BUG=webrtc:1128
Committed: https://crrev.com/f5c7c4790e808d0f1c97b75ae10fb008659ecd02
Cr-Commit-Position: refs/heads/master@{#319652}

[hbos_chromium, 2015-03-09 13:10:03]

hbos@chromium.org changed reviewers:
+ magjed@chromium.org

[magjed_chromium, 2015-03-09 13:20:25]

lgtm

https://codereview.chromium.org/990913002/diff/1/content/renderer/media/rtc_v...
File content/renderer/media/rtc_video_decoder.cc (right):

https://codereview.chromium.org/990913002/diff/1/content/renderer/media/rtc_v...
content/renderer/media/rtc_video_decoder.cc:21: #include
"third_party/webrtc/common_video/interface/i420_video_frame.h"
I think you should use "webrtc/video_frame.h" instead (according to pbos).

[hbos_chromium, 2015-03-09 13:29:55]

hbos@chromium.org changed reviewers:
+ jrummell@chromium.org

[hbos_chromium, 2015-03-09 13:29:56]

I'm not changing the #include because that led to "You added one or more
#includes that violate checkdeps rules."

[hbos_chromium, 2015-03-09 13:53:58]

hbos@chromium.org changed reviewers:
+ tommi@chromium.org
- jrummell@chromium.org

[tommi (sloooow) - chröme, 2015-03-09 13:58:17]

Can you add a bug reference?  E.g.

BUG=webrtc:1128

[tommi (sloooow) - chröme, 2015-03-09 13:58:34]

other than that, lgtm.

[hbos_chromium, 2015-03-09 15:09:19]

The CQ bit was checked by hbos@chromium.org

[commit-bot: I haz the power, 2015-03-09 15:09:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/990913002/1

[commit-bot: I haz the power, 2015-03-09 16:37:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-09 16:38:51]

Patchset 1 (id:??) landed as
https://crrev.com/f5c7c4790e808d0f1c97b75ae10fb008659ecd02
Cr-Commit-Position: refs/heads/master@{#319652}