From 491249108eaea8f2917da27fd5aaef142c1e3a3d Mon Sep 17 00:00:00 2001 From: Tony Ganch Date: Thu, 24 Aug 2017 16:27:54 +0200 Subject: [PATCH] Fix WebLocalFrame API usage Related CLs: https://chromium-review.googlesource.com/c/chromium/src/+/506500 https://chromium-review.googlesource.com/c/chromium/src/+/542658 --- .../renderer/printing/print_web_view_helper.cc | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/chromium_src/chrome/renderer/printing/print_web_view_helper.cc b/chromium_src/chrome/renderer/printing/print_web_view_helper.cc index bbdca2d18a..744b263ae2 100644 --- a/chromium_src/chrome/renderer/printing/print_web_view_helper.cc +++ b/chromium_src/chrome/renderer/printing/print_web_view_helper.cc @@ -525,10 +525,9 @@ void PrepareFrameAndViewForPrint::CopySelection( blink::WebView::Create(this, blink::kWebPageVisibilityStateVisible); owns_web_view_ = true; content::RenderView::ApplyWebPreferences(prefs, web_view); - blink::WebLocalFrame* main_frame = blink::WebLocalFrame::Create( - blink::WebTreeScopeType::kDocument, this, nullptr, nullptr); - web_view->SetMainFrame(main_frame); - blink::WebFrameWidget::Create(this, web_view, main_frame); + blink::WebLocalFrame* main_frame = blink::WebLocalFrame::CreateMainFrame( + web_view, this, nullptr, nullptr); + blink::WebFrameWidget::Create(this, main_frame); frame_.Reset(web_view->MainFrame()->ToWebLocalFrame()); node_to_print_.Reset(); @@ -558,9 +557,8 @@ blink::WebLocalFrame* PrepareFrameAndViewForPrint::CreateChildFrame( blink::WebSandboxFlags sandbox_flags, const blink::WebParsedFeaturePolicy& container_policy, const blink::WebFrameOwnerProperties& frame_owner_properties) { - blink::WebLocalFrame* frame = blink::WebLocalFrame::Create( - scope, this, nullptr, nullptr); - parent->AppendChild(frame); + blink::WebLocalFrame* frame = parent->CreateLocalChild( + scope, this, nullptr); return frame; }