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

Unified Diff: Source/core/rendering/RenderInline.cpp

Issue 788073004: Replace RenderFullscreen with top layer - Take II (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated after review comments. Created 6 years 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/core/rendering/RenderInline.cpp
diff --git a/Source/core/rendering/RenderInline.cpp b/Source/core/rendering/RenderInline.cpp
index f1fd1a803fd1de26b010b0c7621623a32f049f36..ee98d5ca2be8470631628101abf544ef7ece6ccf 100644
--- a/Source/core/rendering/RenderInline.cpp
+++ b/Source/core/rendering/RenderInline.cpp
@@ -34,7 +34,6 @@
#include "core/rendering/InlineTextBox.h"
#include "core/rendering/RenderBlock.h"
#include "core/rendering/RenderFlowThread.h"
-#include "core/rendering/RenderFullScreen.h"
#include "core/rendering/RenderGeometryMap.h"
#include "core/rendering/RenderLayer.h"
#include "core/rendering/RenderTheme.h"
@@ -382,17 +381,6 @@ void RenderInline::splitInlines(RenderBlock* fromBlock, RenderBlock* toBlock,
{
ASSERT(isDescendantOf(fromBlock));
- // If we're splitting the inline containing the fullscreened element,
- // |beforeChild| may be the renderer for the fullscreened element. However,
- // that renderer is wrapped in a RenderFullScreen, so |this| is not its
- // parent. Since the splitting logic expects |this| to be the parent, set
- // |beforeChild| to be the RenderFullScreen.
- if (Fullscreen* fullscreen = Fullscreen::fromIfExists(document())) {
- const Element* fullScreenElement = fullscreen->webkitCurrentFullScreenElement();
- if (fullScreenElement && beforeChild && beforeChild->node() == fullScreenElement)
- beforeChild = fullscreen->fullScreenRenderer();
- }
-
// FIXME: Because splitting is O(n^2) as tags nest pathologically, we cap the depth at which we're willing to clone.
// There will eventually be a better approach to this problem that will let us nest to a much
// greater depth (see bugzilla bug 13430) but for now we have a limit. This *will* result in

Powered by Google App Engine
This is Rietveld 408576698