Consider merging https://chromium.googlesource.com/chromium/blink/+/88f7b61ee65840ec235b0ceae317b23b18a9a627
Trying this in https://github.com/WebKit/WebKit/pull/4364
Created attachment 462376 [details] WIP Patch
Created attachment 462377 [details] WIP Patch
Created attachment 462378 [details] WIP Patch
I tried to land this via PR: https://github.com/WebKit/WebKit/pull/4364 but I was not successful compared to Don and had following build failures: ./loader/FrameLoader.cpp:4264:59: error: no member named 'DisplayingInitialEmptyDocumentPostCommit' in 'WebCore::FrameLoaderStateMachine'; did you mean 'DisplayingInitialEmptyDocument'? Further based on the result from Don, there are crashes and blood bath with this change and there might be more to fix than just doing merge. From the benefits perspective, I think @rniwa can pitch in better but if there is a need to keep this bug open or invest in this simplification, we can. Else we can close this and move on. Thanks!