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

Unified Diff: sky/engine/core/rendering/compositing/CompositedLayerMapping.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
Index: sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
diff --git a/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp b/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
index 8caa7625c9ec87274f477bc303d58471c75e7349..ce4e5c7536fdca0c89321305b6a2ad1a67f3efc8 100644
--- a/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
+++ b/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
@@ -41,7 +41,6 @@
#include "core/rendering/FilterEffectRenderer.h"
#include "core/rendering/RenderImage.h"
#include "core/rendering/RenderLayerStackingNodeIterator.h"
-#include "core/rendering/RenderVideo.h"
#include "core/rendering/RenderView.h"
#include "core/rendering/compositing/RenderLayerCompositor.h"
#include "core/rendering/style/KeyframeList.h"
@@ -62,9 +61,7 @@ static IntRect contentsRect(const RenderObject* renderer)
if (!renderer->isBox())
return IntRect();
- return renderer->isVideo() ?
- toRenderVideo(renderer)->videoBox() :
- pixelSnappedIntRect(toRenderBox(renderer)->contentBoxRect());
+ return pixelSnappedIntRect(toRenderBox(renderer)->contentBoxRect());
}
static IntRect backgroundRect(const RenderObject* renderer)
@@ -1637,10 +1634,6 @@ bool CompositedLayerMapping::containsPaintedContent() const
return false;
RenderObject* renderObject = renderer();
- // FIXME: we could optimize cases where the image, video or canvas is known to fill the border box entirely,
- // and set background color on the layer in that case, instead of allocating backing store and painting.
- if (renderObject->isVideo() && toRenderVideo(renderer())->shouldDisplayVideo())
- return m_owningLayer.hasBoxDecorationsOrBackground();
if (m_owningLayer.hasVisibleBoxDecorations())
return true;
« no previous file with comments | « sky/engine/core/rendering/RenderVideo.cpp ('k') | sky/engine/core/rendering/compositing/RenderLayerCompositor.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698