|
|
Created:
4 years, 3 months ago by loyso (OOO) Modified:
4 years, 3 months ago CC:
chromium-reviews, darin-cc_chromium.org, jam, mlamouri+watch-content_chromium.org, piman+watch_chromium.org Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionContent Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap.
LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS).
IF FrameSwapMessageQueue::DidActivate happens during that frame
THEN
the outputSurface will NOT take care of the Drain+Send.
AND
QueueMessageSwapPromise::DidNotSwap will not send the
MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back.
As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers.
PluginPowerSaverBrowserTest::VerifySnapshot hangs forever.
BUG=599484
Committed: https://crrev.com/260a8f74a7f1230c3252f2640ce7185981d8aef4
Cr-Commit-Position: refs/heads/master@{#417162}
Patch Set 1 #Patch Set 2 : Reparent. #Patch Set 3 : Fix QueueMessageSwapPromiseTest.VisualStateSwapPromiseDidActivate #
Messages
Total messages: 28 (21 generated)
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. BUG=599484 ==========
loyso@chromium.org changed reviewers: + enne@chromium.org
The CQ bit was checked by loyso@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail on frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ==========
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail on frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail on frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ==========
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail on frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ==========
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened then QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened during that frame THEN QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ==========
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened during that frame THEN QueueMessageSwapPromise::DidNotSwap wouldn't send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message so the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ==========
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). If FrameSwapMessageQueue::DidActivate happened during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). IF FrameSwapMessageQueue::DidActivate happens during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ==========
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by loyso@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). IF FrameSwapMessageQueue::DidActivate happens during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). IF FrameSwapMessageQueue::DidActivate happens during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ==========
enne@chromium.org changed reviewers: + jbauman@chromium.org, piman@chromium.org
enne@chromium.org changed reviewers: - jbauman@chromium.org
I'm happy to review this if needed, but am going to defer to piman who reviewed the original frame swap message queue.
piman@, PTAL?
lgtm
The CQ bit was checked by loyso@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Committed patchset #3 (id:40001)
Message was sent while issue was closed.
Description was changed from ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). IF FrameSwapMessageQueue::DidActivate happens during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 ========== to ========== Content Renderer FrameSwapMessageQueue: Fix DidActivate with DidNotSwap. LayerTreeHostImpl::SwapBuffers may fail if frame.has_no_damage (SwapPromise::SWAP_FAILS). IF FrameSwapMessageQueue::DidActivate happens during that frame THEN the outputSurface will NOT take care of the Drain+Send. AND QueueMessageSwapPromise::DidNotSwap will not send the MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE message back. As a result, the callback inserted with RenderFrameHostImpl::InsertVisualStateCallback never triggers. PluginPowerSaverBrowserTest::VerifySnapshot hangs forever. BUG=599484 Committed: https://crrev.com/260a8f74a7f1230c3252f2640ce7185981d8aef4 Cr-Commit-Position: refs/heads/master@{#417162} ==========
Message was sent while issue was closed.
Patchset 3 (id:??) landed as https://crrev.com/260a8f74a7f1230c3252f2640ce7185981d8aef4 Cr-Commit-Position: refs/heads/master@{#417162} |