Commit 58b66459 authored by Dave Tapuska's avatar Dave Tapuska Committed by Commit Bot

Clarify and simplify some code post review

Address post review comments from danakj after
https://chromium-review.googlesource.com/c/chromium/src/+/2044764
was submitted.

BUG=1046424

Change-Id: I8331fceb29ff12d2ec9eda28df9f8b147811de27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062758Reviewed-by: default avatardanakj <danakj@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#742275}
parent 4be47420
......@@ -2031,13 +2031,11 @@ WebLocalFrameImpl* WebViewImpl::MainFrameImpl() const {
void WebViewImpl::DidAttachLocalMainFrame() {
DCHECK(MainFrameImpl());
MainFrameImpl()->GetFrame()->WasAttachedAsLocalMainFrame();
LocalFrame* local_frame = MainFrameImpl()->GetFrame();
local_frame->WasAttachedAsLocalMainFrame();
MainFrameImpl()
->GetFrame()
->GetRemoteNavigationAssociatedInterfaces()
->GetInterface(
local_main_frame_host_remote_.BindNewEndpointAndPassReceiver());
local_frame->GetRemoteNavigationAssociatedInterfaces()->GetInterface(
local_main_frame_host_remote_.BindNewEndpointAndPassReceiver());
if (does_composite_) {
WebWidgetClient* widget_client =
......
......@@ -705,7 +705,8 @@ class CORE_EXPORT WebViewImpl final : public WebView,
Persistent<ResizeViewportAnchor> resize_viewport_anchor_;
// Handle to the local main frame host.
// Handle to the local main frame host. Only valid when the MainFrame is
// local.
mojo::AssociatedRemote<mojom::blink::LocalMainFrameHost>
local_main_frame_host_remote_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment