Cleanup RemotePlatformLayer when disconnecting RemoteFrame.

Source/core/rendering/RenderPart.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Simon Hausmann <hausmann@kde.org>
  *           (C) 2000 Stefan Schimanski (1Stein@gmx.de)
  * Copyright (C) 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 114 matching lines @@
     // renderer and the plugin has a layer, then we need a layer. Second, if this is
     // a renderer with a contentDocument and that document needs a layer, then we need
     // a layer.
     if (widget() && widget()->isPluginView() && toPluginView(widget())->platformLayer())
         return true;
 
     if (!node() || !node()->isFrameOwnerElement())
         return false;
 
     HTMLFrameOwnerElement* element = toHTMLFrameOwnerElement(node());
-    if (element->contentFrame() && element->contentFrame()->remotePlatformLayer())
+    if (element->contentFrame() && element->contentFrame()->isRemoteFrame())
         return true;
 
     if (Document* contentDocument = element->contentDocument()) {
         if (RenderView* view = contentDocument->renderView())
             return view->usesCompositing();
     }
 
     return false;
 }
 
@@ skipping 13 matching lines @@
     if ((inside || result.isRectBasedTest()) && !hadResult && result.innerNode() == node())
         result.setIsOverWidget(contentBoxRect().contains(result.localPoint()));
     return inside;
 }
 
 bool RenderPart::nodeAtPoint(const HitTestRequest& request, HitTestResult& result, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction action)
 {
     if (!widget() || !widget()->isFrameView() || !request.allowsChildFrameContent())
         return nodeAtPointOverWidget(request, result, locationInContainer, accumulatedOffset, action);
 
-    // FIXME: Until RemoteFrames use RemoteFrameViews, we need an explicit check here.
-    if (toFrameView(widget())->frame().isRemoteFrameTemporary())

[kenrb, 2015/01/23 18:31:02]

I am surprised if this does not break input events when running with
--site-per-process. The comment is stale because RemoteFrames now have
RemoteFrameViews, but don't we need to handle the case where widget() is a
RemoteFrameView below? Line 173 should crash as an invalid downcast.

[lfg, 2015/02/03 03:43:46]

RemoteFrameViews return false to isFrameView(), so they should be handled in the
if statement above this.

-        return nodeAtPointOverWidget(request, result, locationInContainer, accumulatedOffset, action);
-
     FrameView* childFrameView = toFrameView(widget());
     RenderView* childRoot = childFrameView->renderView();
 
     if (visibleToHitTestRequest(request) && childRoot) {
         LayoutPoint adjustedLocation = accumulatedOffset + location();
         LayoutPoint contentOffset = LayoutPoint(borderLeft() + paddingLeft(), borderTop() + paddingTop()) - LayoutSize(childFrameView->scrollOffset());
         HitTestLocation newHitTestLocation(locationInContainer, -adjustedLocation - contentOffset);
         HitTestRequest newHitTestRequest(request.type() | HitTestRequest::ChildFrameHitTest);
         HitTestResult childFrameResult(newHitTestLocation);
 
@@ skipping 146 matching lines @@
     if (widget->frameRect() == newFrame)
         return false;
 
     RefPtrWillBeRawPtr<RenderPart> protector(this);
     RefPtrWillBeRawPtr<Node> protectedNode(node());
     widget->setFrameRect(newFrame);
     return widget->frameRect().size() != newFrame.size();
 }
 
 }