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

Unified Diff: sky/engine/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp

Issue 689373003: Remove most of the media stack. (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/GraphicsLayerTreeBuilder.cpp
diff --git a/sky/engine/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp b/sky/engine/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
index 93f0d997108a761f31576789152d863f27d9fee4..6d3745e326fa86bf663de055d88af38f21d502c4 100644
--- a/sky/engine/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
+++ b/sky/engine/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
@@ -27,7 +27,6 @@
#include "config.h"
#include "core/rendering/compositing/GraphicsLayerTreeBuilder.h"
-#include "core/html/HTMLMediaElement.h"
#include "core/rendering/RenderLayer.h"
#include "core/rendering/RenderView.h"
#include "core/rendering/compositing/CompositedLayerMapping.h"
@@ -43,16 +42,6 @@ GraphicsLayerTreeBuilder::~GraphicsLayerTreeBuilder()
{
}
-static bool shouldAppendLayer(const RenderLayer& layer)
-{
- if (!RuntimeEnabledFeatures::overlayFullscreenVideoEnabled())
- return true;
- Node* node = layer.renderer()->node();
- if (node && isHTMLMediaElement(*node) && toHTMLMediaElement(node)->isFullscreen())
- return false;
- return true;
-}
-
void GraphicsLayerTreeBuilder::rebuild(RenderLayer& layer, AncestorInfo info)
{
// Make the layer compositing if necessary, and set up clipping and content layers.
@@ -114,8 +103,7 @@ void GraphicsLayerTreeBuilder::rebuild(RenderLayer& layer, AncestorInfo info)
}
}
- if (shouldAppendLayer(layer))
- info.childLayersOfEnclosingCompositedLayer->append(currentCompositedLayerMapping->childForSuperlayers());
+ info.childLayersOfEnclosingCompositedLayer->append(currentCompositedLayerMapping->childForSuperlayers());
}
if (layer.scrollParent()

Powered by Google App Engine
This is Rietveld 408576698