bcarrella
/
cef
mirror of https://bitbucket.org/chromiumembedded/cef
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

chrome: Fix CHECK when switching from kPrerendering to kActive 

In this case |render_frame_| is not bound, |render_frame_host_| is
non-nullptr and |render_frame_host| equals |render_frame_host_| in
CefFrameHostImpl::MaybeReAttach when called from
CefBrowserInfo::FrameHostStateChanged.
This commit is contained in:
Marshall Greenblatt 2024-06-18 13:49:17 -04:00
parent 3b98dbda79
commit 5afde67593
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
libcef/browser/frame_host_impl.cc
View File

@ -559,11 +559,10 @@ void CefFrameHostImpl::MaybeReAttach(
// Should not be called for temporary frames. // Should not be called for temporary frames.
CHECK(!is_temporary()); CHECK(!is_temporary());
if (require_detached) { // If |require_detached| then we expect that Detach() was called previously.
// We expect that Detach() was called previously. CHECK(!require_detached || !render_frame_.is_bound());
CHECK(!render_frame_.is_bound());
CHECK(!render_frame_host_); if (render_frame_host_) {
} else if (render_frame_host_) {
// Intentionally not clearing |queued_renderer_actions_|, as we may be // Intentionally not clearing |queued_renderer_actions_|, as we may be
// changing RFH during initial browser navigation. // changing RFH during initial browser navigation.
VLOG(1) << GetDebugString() VLOG(1) << GetDebugString()