Commit 26d3c421 authored by behdad's avatar behdad Committed by Chromium LUCI CQ

Minor fix to FrameSorter

The crash reported in bug might have been caused by compiler deducing auto
to refer to a BeginFrameArgs rather than a refference to BeginFrameArgs.
So this change is going to fix this.

Bug: 1153583
Change-Id: I2943230dc92b09ce3e352aff77d03a087e00b985
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2566710Reviewed-by: default avatarJonathan Ross <jonross@chromium.org>
Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#832331}
parent ccdcc502
...@@ -114,7 +114,7 @@ void FrameSorter::FlushFrames() { ...@@ -114,7 +114,7 @@ void FrameSorter::FlushFrames() {
DCHECK(!pending_frames_.empty()); DCHECK(!pending_frames_.empty());
size_t flushed_count = 0; size_t flushed_count = 0;
while (!pending_frames_.empty()) { while (!pending_frames_.empty()) {
const auto first = pending_frames_.front(); const auto& first = pending_frames_.front();
auto& frame_state = frame_states_[first.frame_id]; auto& frame_state = frame_states_[first.frame_id];
if (!frame_state.IsComplete()) if (!frame_state.IsComplete())
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment