With this change if flutter tools provides isolateId parameter, then VM will run the handler on that isolate. If no isolateId is provided, VM will continue running handlers as if they were registered wit Dart_RegisterRootServiceRequestCallback.

This goes towards fixing https://github.com/flutter/flutter/issues/17434

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

iOS does not allow usage of OpenGL ES APIs when the app has been moved to the
background.  With this change, the shell will wait until pending IO thread tasks
complete and the Skia unref queue is drained before proceeding with shutdown.

See https://github.com/flutter/flutter/issues/17511

Update engine for removal of Dart_ReadKernelBinary and refactored build targets.

Auto-roller completed checks. Merging.

This reverts commit 755dbeef.

Update engine for removal of Dart_ReadKernelBinary and refactored build targets.

The Fuchsia bots seem to reference this target to figure out the root_out_directory https://fuchsia.googlesource.com/build/+/master/dart/dart_test.gni#67. Note that the presence of the flutter_tester binary location itself is not depended on. Instead, the target is used to infer the directory containing the flutter_tester after a build.

Patching the //build repository in Fuchsia would mean making the tree red till a //topaz patch lands. To avoid doing this, we add back the missing reference.

This allows applications to start a Flutter engine instance during app startup
and keep it running throughout the app process' lifetime.

FlutterActivity subclasses can override createFlutterNativeView to provide a
preinitialized FlutterNativeView instance and override retainFlutterNativeView
to signal that the FlutterNativeView should be kept alive when the activity
is destroyed.

Auto-roller completed checks. Merging.

s/`[red]`/_correct color_.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

Auto-roller completed checks. Merging.

This removes the Application::Delegate interface and instead uses a closure that captures the task runner. The interface was expected to be more complicated than it turned out to be.

Follows the same pattern followed for termination in response to isolate shutdown.

Instead, keep it around to service more requests. There may be a race between runner shutdown and request for new applications.

Auto-roller completed checks. Merging.

Fixes https://github.com/flutter/flutter/issues/17497

If a line is right justified, then remove any trailing whitespace from the
text range given to Minikin.  Right justification shifts the line's glyphs
by the layout advance computed by Minikin, and this advance should exclude
whitespace so that the last visible character will be flush with the right
margin.

Also exclude trailing whitespace from center justified lines.

Fixes https://github.com/flutter/flutter/issues/17502
Fixes https://github.com/flutter/flutter/issues/16333

Auto-roller completed checks. Merging.

The GPU thread calls into Java via JNI in order to update external textures.

Fixes https://github.com/flutter/flutter/issues/17329

In introduced a regression in text layout: https://github.com/flutter/flutter/issues/17502