Fold WebMediaPlayerImpl::Initialize() into the constructor.

OverrideCreateWebMediaPlayer() was simplified to return a pointer instead of relying on a bool+out-param since unlike OverrideCreatePlugin() we don't require blocking creation of media players.

BUG=109958


Review URL: http://codereview.chromium.org/9187045

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117607 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 9 insertions, 11 deletions

diff --git a/webkit/support/webkit_support.cc b/webkit/support/webkit_support.cc
index fd46091..267e24c 100644
--- a/webkit/support/webkit_support.cc
+++ b/webkit/support/webkit_support.cc
@@ -316,17 +316,15 @@ WebKit::WebMediaPlayer* CreateMediaPlayer(
   scoped_ptr<media::FilterCollection> collection(
       new media::FilterCollection());
 
-  webkit_media::WebMediaPlayerImpl* media_player =
-      new webkit_media::WebMediaPlayerImpl(
-          client,
-          base::WeakPtr<webkit_media::WebMediaPlayerDelegate>(),
-          collection.release(),
-          NULL,
-          message_loop_factory.release(),
-          media_stream_client,
-          new media::MediaLog());
-  media_player->Initialize(frame);
-  return media_player;
+  return new webkit_media::WebMediaPlayerImpl(
+      frame,
+      client,
+      base::WeakPtr<webkit_media::WebMediaPlayerDelegate>(),
+      collection.release(),
+      NULL,
+      message_loop_factory.release(),
+      media_stream_client,
+      new media::MediaLog());
 #endif
 }