Remove old createMediaPlayer method from WebFrameClient implementations

Remove old createMediaPlayer method from WebFrameClient implementations

Now that the blink has shifted to new createMediaPlayer method,
removing the old method from all the WebFrameClient implementations.
This will allow blink to separate the WebMediaPlayerEncryptedMediaClient
from WebMediaPlayerClient and remove the MediaPlayer and MediaPlayerClient
interfaces which is the ultimate goal of the mentioned bugs.

BUG=350571, 310466
Committed: https://crrev.com/9d1423af551de300783135062156069539dc57e0
Cr-Commit-Position: refs/heads/master@{#338825}

[Srirama, 2015-07-14 14:40:55]

srirama.m@samsung.com changed reviewers:
+ ddorwin@chromium.org, philipj@opera.com

[Srirama, 2015-07-14 14:40:56]

PTAL.
This is the last intermediate CL and i can goto
https://codereview.chromium.org/1055503002/ after this.

[philipj_slow, 2015-07-14 14:49:54]

Neither ddorwin nor I are owners of these files, but non-owner LGTM.

[Srirama, 2015-07-14 15:08:28]

On 2015/07/14 14:49:54, philipj wrote:
> Neither ddorwin nor I are owners of these files, but non-owner LGTM.

Thanks Philip, once ddorwin also approves, i will add jam@ for owners review.

[ddorwin, 2015-07-14 18:25:10]

LGTM % removal of includes added in https://codereview.chromium.org/1133033003/.

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
File components/html_viewer/frame.cc (right):

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
components/html_viewer/frame.cc:36: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
Remove now, right?

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
File components/html_viewer/html_document.cc (right):

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
components/html_viewer/html_document.cc:36: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
ditto

https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
content/renderer/render_frame_impl.cc:119: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
ditto

[Srirama, 2015-07-14 18:36:59]

On 2015/07/14 18:25:10, ddorwin wrote:
> LGTM % removal of includes added in
https://codereview.chromium.org/1133033003/.
> 
>
https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
> File components/html_viewer/frame.cc (right):
> 
>
https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
> components/html_viewer/frame.cc:36: #include
> "third_party/WebKit/public/platform/WebMediaPlayerClient.h"
> Remove now, right?
> 
>
https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
> File components/html_viewer/html_document.cc (right):
> 
>
https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
> components/html_viewer/html_document.cc:36: #include
> "third_party/WebKit/public/platform/WebMediaPlayerClient.h"
> ditto
> 
>
https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
> File content/renderer/render_frame_impl.cc (right):
> 
>
https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
> content/renderer/render_frame_impl.cc:119: #include
> "third_party/WebKit/public/platform/WebMediaPlayerClient.h"
> ditto

Hmm, i think i can remove now. I was actually thinking that i will have to wait
for final blink patch to land, but not needed i think. I will remove these
before landing.

[Srirama, 2015-07-14 18:38:46]

srirama.m@samsung.com changed reviewers:
+ jam@chromium.org

[Srirama, 2015-07-14 18:38:46]

jam@: Owner's review please. I will address ddorwin's comments before landing
the patch tomorrow.

[jam, 2015-07-14 20:18:43]

lgtm

[Srirama, 2015-07-15 06:14:25]

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
File components/html_viewer/frame.cc (right):

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/fram...
components/html_viewer/frame.cc:36: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
On 2015/07/14 18:25:10, ddorwin wrote:
> Remove now, right?

Done.

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
File components/html_viewer/html_document.cc (right):

https://codereview.chromium.org/1234573004/diff/1/components/html_viewer/html...
components/html_viewer/html_document.cc:36: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
On 2015/07/14 18:25:10, ddorwin wrote:
> ditto

Done.

https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/1234573004/diff/1/content/renderer/render_fra...
content/renderer/render_frame_impl.cc:119: #include
"third_party/WebKit/public/platform/WebMediaPlayerClient.h"
On 2015/07/14 18:25:10, ddorwin wrote:
> ditto

Done.

[Srirama, 2015-07-15 06:14:31]

The CQ bit was checked by srirama.m@samsung.com

[Srirama, 2015-07-15 06:14:32]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com,
jam@chromium.org, ddorwin@chromium.org
Link to the patchset: https://codereview.chromium.org/1234573004/#ps20001
(title: "removed redundant include")

[commit-bot: I haz the power, 2015-07-15 06:14:38]

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

[commit-bot: I haz the power, 2015-07-15 06:18:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-15 06:19:52]

Patchset 2 (id:??) landed as
https://crrev.com/9d1423af551de300783135062156069539dc57e0
Cr-Commit-Position: refs/heads/master@{#338825}