Remove Width and Height of The Video Content

Remove Width and Height of The Video Content

The width and height in CastRtpPayloadParams is deprecated. Removed them.

BUG=464607
Committed: https://crrev.com/36574c07d51d0b8775df538e8f2b7e84b8a2de32
Cr-Commit-Position: refs/heads/master@{#320054}

[Nico, 2015-03-03 05:45:52]

thakis@chromium.org changed reviewers:
+ miu@chromium.org

[Nico, 2015-03-03 05:45:53]

+miu (author of the TODO) to review

[miu, 2015-03-04 02:09:29]

For completeness, please remove the width/height from the IDL file as well:
chrome/common/extensions/api/cast_streaming_rtp_stream.idl

[zhuoyu.qian, 2015-03-04 02:19:40]

On 2015/03/04 02:09:29, miu wrote:
> For completeness, please remove the width/height from the IDL file as well:
> chrome/common/extensions/api/cast_streaming_rtp_stream.idl

Done.

[miu, 2015-03-04 02:23:22]

miu@chromium.org changed reviewers:
- thakis@chromium.org

[miu, 2015-03-04 02:23:22]

lgtm.  You'll need to get an lgtm from an owner in
chrome/common/extensions/OWNERS for the .idl file change.

Thanks for the clean-up!  :)

[zhuoyu.qian, 2015-03-04 02:48:48]

zhuoyu.qian@samsung.com changed reviewers:
+ finnur@chromium.org, xhwang@chromium.org

[zhuoyu.qian, 2015-03-04 02:51:35]

Hi xhwang: I'd like you to take a look at
  chrome/renderer/media/cast_rtp_stream.cc
  chrome/renderer/media/cast_rtp_stream.h

Hi finnur: I'd like you to take a look at
  chrome/common/extensions/api/cast_streaming_rtp_stream.idl
  chrome/renderer/extensions/cast_streaming_native_handler.cc

[xhwang, 2015-03-04 18:03:44]

xhwang@chromium.org changed reviewers:
+ hclam@chromium.org
- xhwang@chromium.org

[xhwang, 2015-03-04 18:03:44]

You need a per-file owner for cast-*:

see: src/chrome/renderer/media/OWNERS

s/xhwang/hclam

[Finnur, 2015-03-05 10:32:04]

LGTM, one nit.

https://codereview.chromium.org/973753002/diff/20001/chrome/common/extensions...
File chrome/common/extensions/api/cast_streaming_rtp_stream.idl (right):

https://codereview.chromium.org/973753002/diff/20001/chrome/common/extensions...
chrome/common/extensions/api/cast_streaming_rtp_stream.idl:1: // Copyright 2013
The Chromium Authors. All rights reserved.
Not related to this file, but your CL description has an empty BUG=

[zhuoyu.qian, 2015-03-06 01:21:47]

On 2015/03/05 10:32:04, Finnur wrote:
> LGTM, one nit.
> 
>
https://codereview.chromium.org/973753002/diff/20001/chrome/common/extensions...
> File chrome/common/extensions/api/cast_streaming_rtp_stream.idl (right):
> 
>
https://codereview.chromium.org/973753002/diff/20001/chrome/common/extensions...
> chrome/common/extensions/api/cast_streaming_rtp_stream.idl:1: // Copyright
2013
> The Chromium Authors. All rights reserved.
> Not related to this file, but your CL description has an empty BUG=

Done.

[zhuoyu.qian, 2015-03-06 01:22:48]

Hi Alpha: I'd like you to take a look at
  chrome/renderer/media/cast_rtp_stream.cc
  chrome/renderer/media/cast_rtp_stream.h

[Alpha Left Google, 2015-03-10 00:28:42]

The CQ bit was checked by hclam@chromium.org

[Alpha Left Google, 2015-03-10 00:28:43]

lgtm

[commit-bot: I haz the power, 2015-03-10 00:29:42]

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

[commit-bot: I haz the power, 2015-03-10 05:00:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-03-10 05:00:39]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[zhuoyu.qian, 2015-03-11 01:47:53]

The CQ bit was checked by zhuoyu.qian@samsung.com

[commit-bot: I haz the power, 2015-03-11 01:48:27]

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

[zhuoyu.qian, 2015-03-11 02:22:34]

The CQ bit was unchecked by zhuoyu.qian@samsung.com

[zhuoyu.qian, 2015-03-11 05:37:09]

zhuoyu.qian@samsung.com changed reviewers:
+ jcivelli@chromium.org

[zhuoyu.qian, 2015-03-11 05:37:10]

Hi Jay Civelli,
I'd like you to take a look at: 
   chrome/test/data/extensions/api_test/cast_streaming/end_to_end_sender.js
    chrome/test/data/extensions/api_test/cast_streaming/performance.js

[miu, 2015-03-11 06:06:02]

miu@chromium.org changed reviewers:
- jcivelli@chromium.org

[miu, 2015-03-11 06:06:02]

The CQ bit was checked by miu@chromium.org

[miu, 2015-03-11 06:06:03]

lgtm

Test JS changes lgtm.

[miu, 2015-03-11 06:06:03]

The patchset sent to the CQ was uploaded after l-g-t-m from finnur@chromium.org,
hclam@chromium.org
Link to the patchset: https://codereview.chromium.org/973753002/#ps40001 (title:
" ")

[commit-bot: I haz the power, 2015-03-11 06:06:09]

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

[commit-bot: I haz the power, 2015-03-11 07:00:51]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-11 07:01:23]

Patchset 3 (id:??) landed as
https://crrev.com/36574c07d51d0b8775df538e8f2b7e84b8a2de32
Cr-Commit-Position: refs/heads/master@{#320054}