From 01e75761d3713753278a2310715e8590cd152d1f Mon Sep 17 00:00:00 2001 From: Marshall Greenblatt Date: Fri, 18 Oct 2019 14:05:53 +0200 Subject: [PATCH] Remove incorrect DCHECK There are valid cases where |frame_tree_node_id| may not match the node ID reported by the selected RenderFrameHost. --- .../net_service/resource_request_handler_wrapper.cc | 7 ------- 1 file changed, 7 deletions(-) diff --git a/libcef/browser/net_service/resource_request_handler_wrapper.cc b/libcef/browser/net_service/resource_request_handler_wrapper.cc index a0ddb3e34..500f4a230 100644 --- a/libcef/browser/net_service/resource_request_handler_wrapper.cc +++ b/libcef/browser/net_service/resource_request_handler_wrapper.cc @@ -1224,13 +1224,6 @@ void InitOnUIThread( // |frame| may be null for service worker requests. if (frame) { -#if DCHECK_IS_ON() - if (frame_tree_node_id >= 0) { - // Sanity check that we ended up with the expected frame. - DCHECK_EQ(frame_tree_node_id, frame->GetFrameTreeNodeId()); - } -#endif - // May return nullptr for requests originating from guest views. browserPtr = CefBrowserHostImpl::GetBrowserForHost(frame); if (browserPtr) {