Override default implementation of createChildFrame() for print preview.

https://src.chromium.org/viewvc/blink?view=rev&revision=164135 updated
Blink to remove a transitional hack that was implemented to help us
change the way frame lifetimes were managed. However, as a result, Blink
now always assume that createChildFrame() will return a non-null value.
WebFrameClients that expect to ever create child frames must override
this method and frameDetached() if they don't want to crash.

BUG=330994
R=boliu@chromium.org, scottbyer@chromium.org

Review URL: https://codereview.chromium.org/122553002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242941 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 15 insertions, 2 deletions

diff --git a/chrome/renderer/printing/print_web_view_helper.cc b/chrome/renderer/printing/print_web_view_helper.cc
index a53c822..525ef71 100644
--- a/chrome/renderer/printing/print_web_view_helper.cc
+++ b/chrome/renderer/printing/print_web_view_helper.cc
@@ -529,9 +529,13 @@ class PrepareFrameAndViewForPrint : public blink::WebViewClient,
   // blink::WebViewClient override:
   virtual void didStopLoading();
 
-  virtual void CallOnReady();
+  // blink::WebFrameClient override:
+  virtual blink::WebFrame* createChildFrame(blink::WebFrame* parent,
+                                            const blink::WebString& name);
+  virtual void frameDetached(blink::WebFrame* frame);
 
  private:
+  void CallOnReady();
   void ResizeForPrinting();
   void RestoreSize();
   void CopySelection(const WebPreferences& preferences);
@@ -664,6 +668,16 @@ void PrepareFrameAndViewForPrint::didStopLoading() {
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
+blink::WebFrame* PrepareFrameAndViewForPrint::createChildFrame(
+    blink::WebFrame* parent,
+    const blink::WebString& name) {
+  return blink::WebFrame::create(this);
+}
+
+void PrepareFrameAndViewForPrint::frameDetached(blink::WebFrame* frame) {
+  frame->close();
+}
+
 void PrepareFrameAndViewForPrint::CallOnReady() {
   return on_ready_.Run();  // Can delete |this|.
 }
@@ -699,7 +713,6 @@ void PrepareFrameAndViewForPrint::FinishPrinting() {
       DCHECK(!frame->isLoading());
       owns_web_view_ = false;
       web_view->close();
-      frame->close();
     }
   }
   frame_.Reset(NULL);