Commit 9463d54c authored by Helen Li's avatar Helen Li Committed by Commit Bot

Modify handle.is_valid() check in render_frame_host_impl.cc

handle->is_valid() will try to de-reference the handle. This CL changes
->is_valid() to .is_valid() so that the validity check is performed correctly.

Bug: 824144
Change-Id: Ie6f7a5cfa2a7568cdbaceb30e0667d990d1326e3
Reviewed-on: https://chromium-review.googlesource.com/973914Reviewed-by: default avatarKen Rockot <rockot@chromium.org>
Reviewed-by: default avatarJohn Abd-El-Malek <jam@chromium.org>
Commit-Queue: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544851}
parent 850d9723
...@@ -4402,7 +4402,7 @@ void RenderFrameHostImpl::GetInterface( ...@@ -4402,7 +4402,7 @@ void RenderFrameHostImpl::GetInterface(
if (!registry_ || if (!registry_ ||
!registry_->TryBindInterface(interface_name, &interface_pipe)) { !registry_->TryBindInterface(interface_name, &interface_pipe)) {
delegate_->OnInterfaceRequest(this, interface_name, &interface_pipe); delegate_->OnInterfaceRequest(this, interface_name, &interface_pipe);
if (interface_pipe->is_valid() && if (interface_pipe.is_valid() &&
!TryBindFrameInterface(interface_name, &interface_pipe, this)) { !TryBindFrameInterface(interface_name, &interface_pipe, this)) {
GetContentClient()->browser()->BindInterfaceRequestFromFrame( GetContentClient()->browser()->BindInterfaceRequestFromFrame(
this, interface_name, std::move(interface_pipe)); this, interface_name, std::move(interface_pipe));
......
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