This reverts commit c0d5fd23.

* Update packages
* Capture JSON RPC errors that presently get swallowed

## Description

Print actionable warnings if `flutter drive` (where most of our performance benchmarks come from) is run in debug mode and it tries to gather benchmarks using `traceAction`.

## Related Issues

https://github.com/flutter/flutter/issues/30625

## Tests

I added the following tests:
* drive_perf_debug_warning devicelab test

* Add semantic label finders

* add trailing commas on list/map/parameters

* add trailing commas on Invocation with nb of arg>1

* add commas for widget containing widgets

* add trailing commas if instantiation contains trailing comma

* revert bad change

Because we now have 2 separate GPU and UI threads.

These are essentially self-inflicted race conditions. Instead of timeouts we're going to try a more verbose logging mechanism that points out when things are taking a long time.

When setting the Fuchsia logging function, it should happen before any
initialization code, as init can still cause warning/error/info messages
to get printed to logs. Since the default stderr/stdout fd's aren't
correct, this can cause a program to crash for unclear reasons.

This reverts commit 76f70810.

* Remove many timeouts.

These are essentially self-inflicted race conditions. Instead of timeouts we're going to try a more verbose logging mechanism that points out when things are taking a long time.

* Get the attach tests to pass.

* Apply review comments from Todd

* More review comment fixes

* Put back the extended timeouts here now that I know why we have them...

FlutterDriver: allow customizing timeouts using a multiplier

* Future<void> main

* Future<void>.delayed

* prefer_void_to_Null

* address review comments

Adds a Fuchsia compatibility function that allows for connecting to a
specific Isolate by name when running Flutter Driver on a Fuchsia
device.  This will check over multiple Dart VM's in an attempt to find
an Isolate by its name.

See https://github.com/flutter/flutter/pull/19121#issuecomment-419520765

* enable lint unnecessary_new

* fix tests

* fix tests

* fix tests

Fixes the pattern for some TODOs to match our style guide.

(Also, a couple of minor code order fixes.)

* Fix TODO syntax.
* Clarify messages for some timeouts, to aid debugging.
* Increase some other timeouts that were a needlessly short, to reduce sources of flakes.
* Remove some more timeouts that were mostly redundant, to remove complexity.
* Minor style cleanup.
* Remove some dangerous traps (specifically, hide the explicit start/end times in TimedEvent since they shouldn't matter).

- Check memory usage in release builds, not profile.
- Use multiple runs and average the results.

* Revert "Add pub cache, artifacts, pkgs to Cirrus cache (#20080)"

This reverts commit 07e93b38.

* Revert "Reduce noise on the memory benchmarks (#19630)"

This reverts commit 8eb5cb7d.

- Check memory usage in release builds, not profile.
- Use multiple runs and average the results.