Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(257)

Unified Diff: sky/engine/web/WebViewImpl.cpp

Issue 701663002: Remove HTMLVideoElement. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years, 1 month 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
« no previous file with comments | « sky/engine/core/rendering/compositing/RenderLayerCompositor.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/web/WebViewImpl.cpp
diff --git a/sky/engine/web/WebViewImpl.cpp b/sky/engine/web/WebViewImpl.cpp
index e683f066f82b95a69e878f343a5234af5cd2cb2f..1d502b601ba3f6c110a32afc4f029f58f4ab5af9 100644
--- a/sky/engine/web/WebViewImpl.cpp
+++ b/sky/engine/web/WebViewImpl.cpp
@@ -1891,31 +1891,7 @@ void WebViewImpl::setFixedLayoutSize(const WebSize& layoutSize)
void WebViewImpl::performMediaPlayerAction(const WebMediaPlayerAction& action,
const WebPoint& location)
{
- HitTestResult result = hitTestResultForWindowPos(location);
- RefPtr<Node> node = result.innerNonSharedNode();
- if (!isHTMLVideoElement(*node) && !isHTMLAudioElement(*node))
- return;
-
- RefPtr<HTMLMediaElement> mediaElement = static_pointer_cast<HTMLMediaElement>(node);
- switch (action.type) {
- case WebMediaPlayerAction::Play:
- if (action.enable)
- mediaElement->play();
- else
- mediaElement->pause();
- break;
- case WebMediaPlayerAction::Mute:
- mediaElement->setMuted(action.enable);
- break;
- case WebMediaPlayerAction::Loop:
- mediaElement->setLoop(action.enable);
- break;
- case WebMediaPlayerAction::Controls:
- mediaElement->setBooleanAttribute(HTMLNames::controlsAttr, action.enable);
- break;
- default:
- ASSERT_NOT_REACHED();
- }
+ // FIXME(sky): Remove this.
}
WebHitTestResult WebViewImpl::hitTestResultAt(const WebPoint& point)
« no previous file with comments | « sky/engine/core/rendering/compositing/RenderLayerCompositor.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698