Nuked WebFrameImplBase.

third_party/WebKit/public/web/WebFrame.h

 /*
  * Copyright (C) 2009 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 32 matching lines @@
 namespace v8 {
 class Context;
 class Function;
 class Value;
 template <class T>
 class Local;
 }
 
 namespace blink {
 
+class Page;
 class Frame;
 class OpenedFrameTracker;
 class Visitor;
 class WebAssociatedURLLoader;
 struct WebAssociatedURLLoaderOptions;
 class WebDOMEvent;
 class WebData;
 class WebDataSource;
 class WebDocument;
 class WebElement;
-class WebFrameImplBase;
 class WebLocalFrame;
 class WebPerformance;
 class WebRemoteFrame;
 class WebSecurityOrigin;
 class WebSharedWorkerRepositoryClient;
 class WebString;
 class WebURL;
 class WebURLRequest;
 class WebView;
 enum class WebSandboxFlags;
@@ skipping 27 matching lines @@
   };
 
   // Returns the number of live WebFrame objects, used for leak checking.
   BLINK_EXPORT static int InstanceCount();
 
   virtual bool IsWebLocalFrame() const = 0;
   virtual WebLocalFrame* ToWebLocalFrame() = 0;
   virtual bool IsWebRemoteFrame() const = 0;
   virtual WebRemoteFrame* ToWebRemoteFrame() = 0;
 
+  virtual void InitializeCoreFrame(Page&) = 0;
+  // TODO(dcheng): Rename this to coreFrame()? This probably also shouldn't be
+  // const...
+  virtual Frame* GetFrame() const = 0;

[mustaq, 2017/04/21 17:28:53]

I think it makes sense to resolve this TODO in this CL. Wdyt?

+
   BLINK_EXPORT bool Swap(WebFrame*);
 
   // This method closes and deletes the WebFrame. This is typically called by
   // the embedder in response to a frame detached callback to the WebFrame
   // client.
   virtual void Close();
 
   // Called by the embedder when it needs to detach the subtree rooted at this
   // frame.
   BLINK_EXPORT void Detach();
@@ skipping 289 matching lines @@
   // Returns the bounds rect for current selection. If selection is performed
   // on transformed text, the rect will still bound the selection but will
   // not be transformed itself. If no selection is present, the rect will be
   // empty ((0,0), (0,0)).
   virtual WebRect SelectionBoundsRect() const = 0;
 
   // Dumps the layer tree, used by the accelerated compositor, in
   // text form. This is used only by layout tests.
   virtual WebString LayerTreeAsText(bool show_debug_info = false) const = 0;
 
-  virtual WebFrameImplBase* ToImplBase() = 0;
-  // TODO(dcheng): Fix const-correctness issues and remove this overload.
-  virtual const WebFrameImplBase* ToImplBase() const {
-    return const_cast<WebFrame*>(this)->ToImplBase();
-  }
-
   // Returns the frame inside a given frame or iframe element. Returns 0 if
   // the given element is not a frame, iframe or if the frame is empty.
   BLINK_EXPORT static WebFrame* FromFrameOwnerElement(const WebElement&);
 
 #if BLINK_IMPLEMENTATION
   static WebFrame* FromFrame(Frame*);
 
   bool InShadowTree() const { return scope_ == WebTreeScopeType::kShadow; }
 
   static void TraceFrames(Visitor*, WebFrame*);
@@ skipping 25 matching lines @@
   WebFrame* first_child_;
   WebFrame* last_child_;
 
   WebFrame* opener_;
   std::unique_ptr<OpenedFrameTracker> opened_frame_tracker_;
 };
 
 }  // namespace blink
 
 #endif