[WIP] Migrate Fullscreen to use top layer instead of RenderFullScreen

Source/core/rendering/RenderVideo.cpp

 /*
  * Copyright (C) 2007, 2008, 2009, 2010 Apple Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 17 matching lines @@
 #include "core/rendering/RenderVideo.h"
 
 #include "HTMLNames.h"
 #include "core/dom/Document.h"
 #include "core/html/HTMLVideoElement.h"
 #include "core/page/Frame.h"
 #include "core/page/FrameView.h"
 #include "core/page/Page.h"
 #include "core/platform/graphics/MediaPlayer.h"
 #include "core/rendering/PaintInfo.h"
-#include "core/rendering/RenderFullScreen.h"
 
 namespace WebCore {
 
 using namespace HTMLNames;
 
 RenderVideo::RenderVideo(HTMLVideoElement* video)
     : RenderMedia(video)
 {
     setIntrinsicSize(calculateIntrinsicSize());
 }
@@ skipping 211 matching lines @@
 
 bool RenderVideo::supportsAcceleratedRendering() const
 {
     MediaPlayer* p = mediaElement()->player();
     if (p)
         return p->supportsAcceleratedRendering();
 
     return false;
 }
 
-static const RenderBlock* rendererPlaceholder(const RenderObject* renderer)
-{
-    RenderObject* parent = renderer->parent();
-    if (!parent)
-        return 0;
-    
-    RenderFullScreen* fullScreen = parent->isRenderFullScreen() ? toRenderFullScreen(parent) : 0;
-    if (!fullScreen)
-        return 0;
-    
-    return fullScreen->placeholder();
-}
-
-LayoutUnit RenderVideo::offsetLeft() const
-{
-    if (const RenderBlock* block = rendererPlaceholder(this))
-        return block->offsetLeft();
-    return RenderMedia::offsetLeft();
-}
-
-LayoutUnit RenderVideo::offsetTop() const
-{
-    if (const RenderBlock* block = rendererPlaceholder(this))
-        return block->offsetTop();
-    return RenderMedia::offsetTop();
-}
-
-LayoutUnit RenderVideo::offsetWidth() const
-{
-    if (const RenderBlock* block = rendererPlaceholder(this))
-        return block->offsetWidth();
-    return RenderMedia::offsetWidth();
-}
-
-LayoutUnit RenderVideo::offsetHeight() const
-{
-    if (const RenderBlock* block = rendererPlaceholder(this))
-        return block->offsetHeight();
-    return RenderMedia::offsetHeight();
-}
-

[falken, 2013/07/09 12:50:22]

These were added in https://bugs.webkit.org/show_bug.cgi?id=62400

It's unclear why removing them doesn't seem to break any tests.

 } // namespace WebCore