Source/web/FrameLoaderClientImpl.cpp - Issue 1055503002: Eliminate MediaPlayer & MediaPlayerClient abstractions

Unified Diff: Source/web/FrameLoaderClientImpl.cpp

Issue 1055503002: Eliminate MediaPlayer & MediaPlayerClient abstractions (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Patch cleanup and review comments fix Created 5 years, 7 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: Source/web/FrameLoaderClientImpl.cpp

diff --git a/Source/web/FrameLoaderClientImpl.cpp b/Source/web/FrameLoaderClientImpl.cpp

index 802c78b23d51525b29d5d6f8cbe42ed0f3b473a3..679838c3d90aac497c4df086b7d673153e5616b5 100644

--- a/Source/web/FrameLoaderClientImpl.cpp

+++ b/Source/web/FrameLoaderClientImpl.cpp

@@ -42,6 +42,7 @@

#include "core/frame/FrameView.h"

#include "core/frame/Settings.h"

#include "core/html/HTMLAppletElement.h"

+#include "core/html/HTMLMediaElement.h"

#include "core/layout/HitTestResult.h"

#include "core/loader/DocumentLoader.h"

#include "core/loader/FrameLoadRequest.h"

@@ -54,6 +55,7 @@

#include "modules/device_light/DeviceLightController.h"

#include "modules/device_orientation/DeviceMotionController.h"

#include "modules/device_orientation/DeviceOrientationController.h"

+#include "modules/encryptedmedia/HTMLMediaElementEncryptedMedia.h"

#include "modules/gamepad/NavigatorGamepad.h"

#include "modules/serviceworkers/NavigatorServiceWorker.h"

#include "modules/storage/DOMWindowStorageController.h"

@@ -67,6 +69,7 @@

#include "platform/plugins/PluginData.h"

#include "public/platform/Platform.h"

#include "public/platform/WebApplicationCacheHost.h"

+#include "public/platform/WebMediaPlayer.h"

#include "public/platform/WebMimeRegistry.h"

#include "public/platform/WebRTCPeerConnectionHandler.h"

#include "public/platform/WebSecurityOrigin.h"

@@ -790,6 +793,21 @@ PassRefPtrWillBeRawPtr<Widget> FrameLoaderClientImpl::createJavaAppletWidget(

"application/x-java-applet", false, FailOnDetachedPlugin);

}

+PassOwnPtr<WebMediaPlayer> FrameLoaderClientImpl::createWebMediaPlayer(

+ HTMLMediaElement* htmlMediaElement,

+ const WebURL& url)

+ WebLocalFrameImpl* webFrame = WebLocalFrameImpl::fromFrame(

+ htmlMediaElement->document().frame());

+ if (!webFrame || !webFrame->client())

+ return nullptr;

+ HTMLMediaElementEncryptedMedia& encryptedMedia = HTMLMediaElementEncryptedMedia::from(*htmlMediaElement);

+ return adoptPtr(webFrame->client()->createMediaPlayer(webFrame, url, static_cast<WebMediaPlayerClient*>(htmlMediaElement),

ddorwin 2015/05/19 17:01:49 It doesn't look like this was done. I was referrin

+ &encryptedMedia, encryptedMedia.contentDecryptionModule(*htmlMediaElement)));

ObjectContentType FrameLoaderClientImpl::objectContentType(

const KURL& url,

const String& explicitMimeType,

« Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.cpp ('K') | « Source/web/FrameLoaderClientImpl.h ('k') | Source/web/WebKit.cpp » ('j') | no next file with comments »