Fix isLayoutTimerActive for ports that set a minimumLayoutDelay
Created attachment 83081 [details] Patch
Comment on attachment 83081 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=83081&action=review > Source/WebCore/ChangeLog:8 > + No new tests because no new functionality. So we have no way to test that we're yielding between tokens? Do we have any way to detect first paint from JS? Certainly there is in FF, but I'm not sure there is in WebKit. > Source/WebCore/dom/Document.cpp:2160 > + return view() && view()->layoutPending() && minimumLayoutDelay() == m_extraLayoutDelay; Maybe the minimum = extra check needs itself to be a function. Given how strange that reads.
Comment on attachment 83081 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=83081&action=review >> Source/WebCore/ChangeLog:8 >> + No new tests because no new functionality. > > So we have no way to test that we're yielding between tokens? Do we have any way to detect first paint from JS? Certainly there is in FF, but I'm not sure there is in WebKit. It's very difficult to test. It might be observable, but I can't think of how to observe it without races. >> Source/WebCore/dom/Document.cpp:2160 >> + return view() && view()->layoutPending() && minimumLayoutDelay() == m_extraLayoutDelay; > > Maybe the minimum = extra check needs itself to be a function. Given how strange that reads. Yeah, if I were reading this code in the future, it would be pretty mysterious.
Comment on attachment 83081 [details] Patch cq- for functionization.
Created attachment 83737 [details] Patch for landing
I went with a bool "isPendingLayoutImmediate" to make this more clear. We can pull it out to a function if anything else needs it. But I'm not sure proliferation of this condition should be encouraged.
Comment on attachment 83737 [details] Patch for landing Clearing flags on attachment: 83737 Committed r79775: <http://trac.webkit.org/changeset/79775>
All reviewed patches have been landed. Closing bug.
The commit-queue encountered the following flaky tests while processing attachment 83737 [details]: http/tests/websocket/tests/multiple-connections.html bug 53825 (author: abarth@webkit.org) The commit-queue is continuing to process your patch.