Revert of Require WebLocalFrame to be created with a non-null client

third_party/WebKit/public/web/WebFrameClient.h

 /*
  * Copyright (C) 2011, 2012 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 65 matching lines @@
 class WebContentDecryptionModule;
 class WebCookieJar;
 class WebCString;
 class WebDataSource;
 class WebEncryptedMediaClient;
 class WebExternalPopupMenu;
 class WebExternalPopupMenuClient;
 class WebFileChooserCompletion;
 class WebFormElement;
 class WebInstalledAppClient;
-class WebLocalFrame;
 class WebMediaPlayer;
 class WebMediaPlayerClient;
 class WebMediaPlayerEncryptedMediaClient;
 class WebMediaPlayerSource;
 class WebMediaSession;
 class WebMediaStream;
 class WebServiceWorkerProvider;
 class WebPlugin;
 class WebPresentationClient;
 class WebPushClient;
 class WebRTCPeerConnectionHandler;
 class WebScreenOrientationClient;
 class WebString;
 class WebURL;
 class WebURLResponse;
 class WebUserMediaClient;
 class WebWorkerContentSettingsClientProxy;
 struct WebColorSuggestion;
 struct WebConsoleMessage;
 struct WebContextMenuData;
 struct WebPluginParams;
 struct WebPopupMenuInfo;
 struct WebRect;
 struct WebURLError;
 
-class BLINK_EXPORT WebFrameClient {
+class WebFrameClient {
 public:
-    virtual ~WebFrameClient() {}
-
     // Factory methods -----------------------------------------------------
 
     // May return null.
     virtual WebPlugin* createPlugin(WebLocalFrame*, const WebPluginParams&) { return 0; }
 
     // May return null.
     // WebContentDecryptionModule* may be null if one has not yet been set.
     virtual WebMediaPlayer* createMediaPlayer(const WebMediaPlayerSource&, WebMediaPlayerClient*, WebMediaPlayerEncryptedMediaClient*, WebContentDecryptionModule*, const WebString& sinkId) { return 0; }
 
     // May return null.
@@ skipping 26 matching lines @@
     // General notifications -----------------------------------------------
 
     // Indicates if creating a plugin without an associated renderer is supported.
     virtual bool canCreatePluginWithoutRenderer(const WebString& mimeType) { return false; }
 
     // Indicates that another page has accessed the DOM of the initial empty
     // document of a main frame. After this, it is no longer safe to show a
     // pending navigation's URL, because a URL spoof is possible.
     virtual void didAccessInitialDocument() { }
 
-    // Request the creation of a new child frame. Embedders may return nullptr
-    // to prevent the new child frame from being attached. Otherwise, embedders
-    // should create a new WebLocalFrame, insert it into the frame tree, and
-    // return the created frame.
-    virtual WebLocalFrame* createChildFrame(WebLocalFrame* parent, WebTreeScopeType, const WebString& name, const WebString& uniqueName, WebSandboxFlags sandboxFlags, const WebFrameOwnerProperties&) { return nullptr; }
+    // A child frame was created in this frame. This is called when the frame
+    // is created and initialized. Takes the name of the new frame, the parent
+    // frame and returns a new WebFrame. The WebFrame is considered in-use
+    // until frameDetached() is called on it.
+    // Note: If you override this, you should almost certainly be overriding
+    // frameDetached().
+    virtual WebFrame* createChildFrame(WebLocalFrame* parent, WebTreeScopeType, const WebString& name, const WebString& uniqueName, WebSandboxFlags sandboxFlags, const WebFrameOwnerProperties&) { return nullptr; }
 
     // This frame has set its opener to another frame, or disowned the opener
     // if opener is null. See http://html.spec.whatwg.org/#dom-opener.
     virtual void didChangeOpener(WebFrame*) { }
 
     // Specifies the reason for the detachment.
     enum class DetachType { Remove, Swap };
 
-    // This frame has been detached. Embedders should release any resources
-    // associated with this frame. If the DetachType is Remove, the frame should
-    // also be removed from the frame tree; otherwise, if the DetachType is
-    // Swap, the frame is being replaced in-place by WebFrame::swap().
-    virtual void frameDetached(WebLocalFrame*, DetachType);
+    // This frame has been detached from the view, but has not been closed yet.
+    virtual void frameDetached(WebLocalFrame*, DetachType) {}
 
-    // This frame has become focused.
+    // This frame has become focused..
     virtual void frameFocused() { }
 
-    // A provisional load is about to commit.
-    virtual void willCommitProvisionalLoad(WebLocalFrame*) {}
+    // This frame is about to be closed. This is called after frameDetached,
+    // when the document is being unloaded, due to new one committing.
+    virtual void willClose(WebFrame*) { }
 
     // This frame's name has changed.
     virtual void didChangeName(const WebString& name, const WebString& uniqueName) { }
 
     // This frame has set an insecure request policy.
     virtual void didEnforceInsecureRequestPolicy(WebInsecureRequestPolicy) {}
 
     // This frame has been updated to a unique origin, which should be
     // considered potentially trustworthy if
     // |isPotentiallyTrustworthyUniqueOrigin| is true. TODO(estark):
@@ skipping 493 matching lines @@
     {
         return WebPageVisibilityStateVisible;
     }
 
     // Overwrites the given URL to use an HTML5 embed if possible.
     // An empty URL is returned if the URL is not overriden.
     virtual WebURL overrideFlashEmbedWithHTML(const WebURL& url)
     {
         return WebURL();
     }
+
+protected:
+    virtual ~WebFrameClient() { }
 };
 
 } // namespace blink
 
 #endif