commit | 586af386595943856c00e3d78f720dfa3d82d53a | [log] [tgz] |
---|---|---|
author | Sunny Sachanandani <sunnyps@chromium.org> | Tue Jul 24 21:44:38 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Jul 24 21:44:38 2018 |
tree | 98be3a2a4fe35eb27e1663e3ffe57aa63206e009 | |
parent | b9d9251e5823107903c09909cfb47dd3078f02c1 [diff] |
Do not call IsActiveTree() in LayerTreeImpl destructor IsActiveTree() indirectly calls active_tree_.get() in LayerTreeHostImpl. Since std::unique_ptr::reset() nulls out its ptr before deleting the owned object, IsActiveTree() will always be false, and the swap promises will always be broken with ACTIVATION_FAILS. This causes LayerTreeHostTestSwapPromiseDuringCommit to fail when the wait for main frame or commit condition is removed from LayerTreeTest::EndTest(). R=danakj BUG=703263 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I3819f76ab8f576c80faccf8db3ecd7ddb9b5954d Reviewed-on: https://chromium-review.googlesource.com/1145920 Reviewed-by: danakj <danakj@chromium.org> Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#577696}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .