Replace LinkedLists that are used as a queue in android FlutterRenderer with ArrayDeques

[gmackall]

Inspired by #50767.

As their documentation notes, ArrayDeques are generally faster than LinkedList when used as a queue.

Fixes flutter/flutter#143721

Pre-launch Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I read the Tree Hygiene wiki page, which explains my responsibilities.
• I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
• I listed at least one issue that this PR fixes in the description above.
• I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
• I updated/added relevant documentation (doc comments with ///).
• I signed the CLA.
• All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

[jonahwilliams]

I have some notes on benchmarking Android CPU times in https://github.com/flutter/engine/blob/main/impeller/docs/android_cpu_profile.md

As long as it is faster locally I'd consider an NTE

[gmackall]

I have some notes on benchmarking Android CPU times in https://github.com/flutter/engine/blob/main/impeller/docs/android_cpu_profile.md

As long as it is faster locally I'd consider an NTE

Thanks, I got the benchmarking working. Can I ask what app you were benchmarking in flutter/flutter#143721? It looks like in another issue you referenced it being the gallery, but I see we have both
https://github.com/flutter/gallery
and
https://github.com/flutter/flutter/tree/master/dev/integration_tests/flutter_gallery
The first one fails to ever launch for me, while the second one has a bunch of problems when I try to run with --enable-impeller on my emulator. Maybe I need to run it on a device?

[jonahwilliams]

This isn't really impeller specific, you should be able to see this with the Skia backend too. Never benchmark on an emulator, you should always use a real device.

@johnmccutchan has a fullscreen webview example I was using, I don't have a link to it, but any platform view that continually pumps frames should work.

[gmackall]

I finally got around to testing this, (tested this with the video player plugin on @matanlurey's branch that converts to SurfaceProducer)

Saw a similar (though slightly less, from ~20% spent in linked list traversal to ~13% in ArrayDeque traversal) to what was noted in this comment, so I'm going to land this