Revert of Separate RenderView from RenderWidget, part 1: inheritance.

content/renderer/render_frame_impl_browsertest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdint.h>
 
 #include "base/command_line.h"
 #include "base/debug/leak_annotations.h"
 #include "build/build_config.h"
 #include "content/common/frame_messages.h"
@@ skipping 107 matching lines @@
 #define MAYBE_SubframeWidget SubframeWidget
 #define MAYBE_FrameResize FrameResize
 #define MAYBE_FrameWasShown FrameWasShown
 #define MAYBE_FrameWasShownAfterWidgetClose FrameWasShownAfterWidgetClose
 #endif
 
 // Verify that a frame with a RenderFrameProxy as a parent has its own
 // RenderWidget.
 TEST_F(RenderFrameImplTest, MAYBE_SubframeWidget) {
   EXPECT_TRUE(frame_widget());
-  EXPECT_NE(frame_widget(), view_->GetWidget());
+  // We can't convert to RenderWidget* directly, because
+  // it and RenderView are two unrelated base classes
+  // of RenderViewImpl. If a class has multiple base classes,
+  // each base class pointer will be distinct, and direct casts
+  // between unrelated base classes are undefined, even if they share
+  // a common derived class. The compiler has no way in general of
+  // determining the displacement between the two classes, so these
+  // types of casts cannot be implemented in a type safe way.
+  // To overcome this, we make two legal static casts:
+  // first, downcast from RenderView* to RenderViewImpl*,
+  // then upcast from RenderViewImpl* to RenderWidget*.
+  EXPECT_NE(frame_widget(),
+            static_cast<content::RenderWidget*>(
+                static_cast<content::RenderViewImpl*>((view_))));
 }
 
 // Verify a subframe RenderWidget properly processes its viewport being
 // resized.
 TEST_F(RenderFrameImplTest, MAYBE_FrameResize) {
   ResizeParams resize_params;
   gfx::Size size(200, 200);
   resize_params.screen_info = blink::WebScreenInfo();
   resize_params.new_size = size;
   resize_params.physical_backing_size = size;
@@ skipping 75 matching lines @@
   // but serves the purpose of testing the LoFi state logic.
   GetMainRenderFrame()->didCommitProvisionalLoad(
       GetMainRenderFrame()->GetWebFrame(), item, blink::WebStandardCommit);
   EXPECT_FALSE(GetMainRenderFrame()->IsUsingLoFi());
   // The subframe would be deleted here after a cross-document navigation. It
   // happens to be left around in this test because this does not simulate the
   // frame detach.
 }
 
 }  // namespace