Commit ebb8f749 authored by arthursonzogni's avatar arthursonzogni Committed by Commit Bot

Cleanup redundant local variable.

This is a follow-up to haraken@ comment on:
https://chromium-review.googlesource.com/c/chromium/src/+/1533913/33/content/renderer/render_frame_impl.cc#1517

Bug: None
Change-Id: I03a15efd4bca486c7400693ac3cdfe9da32b14fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083532Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#746296}
parent 09d05a61
......@@ -1502,8 +1502,6 @@ void RenderFrameImpl::CreateFrame(
// main frame, as in the case where a navigation to the current process'
// origin replaces a remote main frame (the proxy's web_frame()) with a
// local one.
const bool proxy_is_main_frame = !proxy->web_frame()->Parent();
render_view = proxy->render_view();
render_frame = RenderFrameImpl::Create(
render_view, routing_id, std::move(interface_provider),
......@@ -1516,7 +1514,7 @@ void RenderFrameImpl::CreateFrame(
proxy->web_frame(), replicated_state.frame_policy,
WebString::FromUTF8(replicated_state.name));
// The new |web_frame| is a main frame iff the proxy's frame was.
DCHECK_EQ(proxy_is_main_frame, !web_frame->Parent());
DCHECK_EQ(!proxy->web_frame()->Parent(), !web_frame->Parent());
}
CHECK(render_view);
......
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