Separate RenderView from RenderWidget, part 1: inheritance.

content/renderer/render_frame_impl_browsertest.cc

Index: content/renderer/render_frame_impl_browsertest.cc
diff --git a/content/renderer/render_frame_impl_browsertest.cc b/content/renderer/render_frame_impl_browsertest.cc
index 00456c9a6a09036be6456d2416b701698cd198a0..79b647d5347908ed6b6466906b2ab7c1936bcc01 100644
--- a/content/renderer/render_frame_impl_browsertest.cc
+++ b/content/renderer/render_frame_impl_browsertest.cc
@@ -125,20 +125,7 @@ class RenderFrameTestObserver : public RenderFrameObserver {
 // RenderWidget.
 TEST_F(RenderFrameImplTest, MAYBE_SubframeWidget) {
   EXPECT_TRUE(frame_widget());
-  // 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_))));
+  EXPECT_NE(frame_widget(), view_->GetWidget());
 }
 
 // Verify a subframe RenderWidget properly processes its viewport being