Creating canvas dynamically in webrtc video quality tests

Creating canvas dynamically in webrtc video quality tests

The canvas on which the frame is captured is created dynamically.
References of canvas and the remote video are passed instead of the hard
coded 'Id's.

TEST=out/Release/browser_tests --gtest_filter=WebRtcVideoQualityBrowserTest* --run-manual --ui-test-action-max-timeout=150000

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270726

[amogh.bihani, 2014-05-13 13:23:24]

PTAL.
I have made the changes as suggested in the previous CL.
https://codereview.chromium.org/254803002/
This CL negates some of the changes made in that CL.

Please note that I do not have access to svn.chromium.org hence I do not have
the actual testing videos. I have taken some random y4m and corresponding yuv
file to test. I can't be sure that the results are accurate, however, the test
does seem similar.

https://codereview.chromium.org/280383007/diff/1/chrome/test/data/webrtc/vide...
File chrome/test/data/webrtc/video_extraction.js (left):

https://codereview.chromium.org/280383007/diff/1/chrome/test/data/webrtc/vide...
chrome/test/data/webrtc/video_extraction.js:120: function
setCanvasDimensions_(width, height) {
Now these changes seem futile. :(

https://codereview.chromium.org/280383007/diff/1/chrome/test/data/webrtc/webr...
File chrome/test/data/webrtc/webrtc_video_quality_test.html (right):

https://codereview.chromium.org/280383007/diff/1/chrome/test/data/webrtc/webr...
chrome/test/data/webrtc/webrtc_video_quality_test.html:23: <!--
startFrameCapture() takes 3 parameters:
Sorry, I forgot to make this change(line 23 - 26) in the previous CL.
https://codereview.chromium.org/254803002/

[amogh.bihani, 2014-05-15 09:19:53]

Ping!

[phoglund_chromium, 2014-05-15 09:34:13]

lgtm (sorry for the delay)

[amogh.bihani, 2014-05-15 09:48:08]

On 2014/05/15 09:34:13, phoglund wrote:
> lgtm (sorry for the delay)

Thanks :)

[amogh.bihani, 2014-05-15 09:48:12]

The CQ bit was checked by amogh.bihani@samsung.com

[commit-bot: I haz the power, 2014-05-15 09:50:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/amogh.bihani@samsung.com/280383007/20001

[commit-bot: I haz the power, 2014-05-15 17:42:01]

Change committed as 270726