DescriptionReland "Simplify FrameLoader::Clear(), remove FrameLoader::check_timer"
Clear() is full of strange, old logic. Several clauses are outright unnecessary, or have better-encapsulated homes.
check_timer_ appears to have two uses, one of which is vestigial, the other of which can be better-situated to not require a timer.
BUG=
Review-Url: https://codereview.chromium.org/2878553002
Cr-Commit-Position: refs/heads/master@{#471526}
Committed: https://chromium.googlesource.com/chromium/src/+/2e62c43c2024d820f36302576f89cec7dd46130c
Patch Set 1 #Patch Set 2 : Reorder LocalFrame::Detach, assert !IsLoading() before FrameLoader::Detach #Patch Set 3 : Don't call CheckCompleted in HTMLFrameOwnerElement #
Messages
Total messages: 17 (13 generated)
|