Enable media stream layout test with content_shell.

Enable media stream layout test with content_shell. 

When using content_shell to run media stream layout test, it's needed to use override createMediaPlayer because the media stream is generated by TestMediaStreamClient, not MediaStreamImpl which is used by RenderViewImpl::createMediaPlayer.

This patch is needed by patch in https://bugs.webkit.org/show_bug.cgi?id=113633, and they work together to enable media stream layout test in content_shell.

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

[wjia(left Chromium), 2013-04-02 02:43:50]

PTAL this patch and https://bugs.webkit.org/show_bug.cgi?id=113633.

[jochen (gone - plz use gerrit), 2013-04-02 07:02:20]

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
File content/shell/webkit_test_runner.cc (right):

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
content/shell/webkit_test_runner.cc:62: #include
"webkit/support/webkit_support.h"
content/ must not depend on webkit/support.

You could move the test_media_stream_client to webkit/mocks, and just copy the
CreateMediaPlayer method to this file.

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
File content/shell/webkit_test_runner.h (right):

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
content/shell/webkit_test_runner.h:100: WebKit::WebFrame*,
the chromium coding style requires to list argument names.

[wjia(left Chromium), 2013-04-03 00:44:04]

ptal. I will update WebKit patch. This patch should land before the WebKit
patch. There are couple of TODO's which will be done after WebKit patch is
landed.

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
File content/shell/webkit_test_runner.cc (right):

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
content/shell/webkit_test_runner.cc:62: #include
"webkit/support/webkit_support.h"
On 2013/04/02 07:02:21, jochen wrote:
> content/ must not depend on webkit/support.
> 
> You could move the test_media_stream_client to webkit/mocks, and just copy the
> CreateMediaPlayer method to this file.

Done.

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
File content/shell/webkit_test_runner.h (right):

https://codereview.chromium.org/13159005/diff/19002/content/shell/webkit_test...
content/shell/webkit_test_runner.h:100: WebKit::WebFrame*,
On 2013/04/02 07:02:21, jochen wrote:
> the chromium coding style requires to list argument names.

Done.

[jochen (gone - plz use gerrit), 2013-04-03 07:18:58]

lgtm

https://codereview.chromium.org/13159005/diff/24001/content/shell/webkit_test...
File content/shell/webkit_test_runner.cc (right):

https://codereview.chromium.org/13159005/diff/24001/content/shell/webkit_test...
content/shell/webkit_test_runner.cc:498: if (!test_media_stream_client_.get())
nit. { } required for multi-line if body

[wjia(left Chromium), 2013-04-04 14:21:51]

+jamesr@ for owner stamp on webkit/

https://codereview.chromium.org/13159005/diff/24001/content/shell/webkit_test...
File content/shell/webkit_test_runner.cc (right):

https://codereview.chromium.org/13159005/diff/24001/content/shell/webkit_test...
content/shell/webkit_test_runner.cc:498: if (!test_media_stream_client_.get())
On 2013/04/03 07:18:58, jochen wrote:
> nit. { } required for multi-line if body

Done.

[jamesr, 2013-04-04 17:56:15]

webkit/ lgtm

[wjia(left Chromium), 2013-04-04 18:25:27]

Committed patchset #5 manually as r192373 (presubmit successful).