These are now flagged by the Dart analyzer.

This reverts commit 4126c066.

The LUCI recipe issue should have already been fixed by @christopherfujino 

Try to trigger the LUCI tests with this as the "retry" button is somehow disabled for https://ci.chromium.org/p/flutter/builders/prod/Linux%20Fuchsia/4755 even if I logged in.

This reverts commit 00e75747.

Try to fix the LUCI Fuchsia build.

- Rolls buildroot to pick up
  https://github.com/flutter/buildroot/pull/389
- Adds support to the gn wrapper for setting the traget CPU on Windows

Attempting to generate Windows arm64 GN files will currently fail, due
to lack of arm64 support for an OpenGL test support target, but this
sets the foundation, and makes experimentation easier.

Part of https://github.com/flutter/flutter/issues/62597

Instead of a hand-rolled discriminated union (originally used to avoid a C++17
dependency, which is no longer an issue), implement EncodableValue as a
std::variant. Rather than simply changing the internals, this makes EncodableValue
a minimal std::variant subclass with only a handful of added methodS, replacing
the old IsFoo/FooValue APIs with the standard std::holds_alternative/std::get,
so that plugin code will use a standard-based API rather than a Flutter-specific
API for wrapped values.

This is a breaking change for Windows and GLFW plugins. In the short
term USE_LEGACY_ENCODABLE_VALUE can be set in builds to use the old
version, to separate rolling from updating.

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

The fuchsia code around metrics and sizing was just sending this
information through a side-channel, when the engine already had the
information available. So, delete all of it to make future CLs simpler.

Additionally, the SceneUpdateContext has many unneccesary dependencies
re: metrics and PaintTasks. Break those to make future CLs simpler.

Tested: Ran all unittests and ran workstation on Fuchsia
BUG: 53062, 53063

* Implement Clipboard hasStrings method on iOS

* Test call to hasStrings

* Formatting fixes

* Move test to its own file

* Alphabetical order

* Update licenses

* arguments nil instead of empty dictionary

* Guarantee hasStrings will be true when tested

* Formatting

https://skia.googlesource.com/skia.git/+log/7225788b9070..8e77631a80f0

2020-07-31 borenet@google.com Remove some non-inclusive language in recipes
2020-07-31 scroggo@google.com Remove "sanity"/"insane" to comply with Android's inclusive language guidance
2020-07-31 scroggo@google.com Remove "dummy" to comply with Android's inclusive language guidance
2020-07-31 jdapena@igalia.com GCC: fix type passed to vcvt_f32_f16 to be a float16x4_t in SkVx.h
2020-07-31 scroggo@google.com Remove "Cripple" to comply with Android's inclusive language guidance
2020-07-31 fmalita@chromium.org Fix path builder initial gen ID
2020-07-31 ethannicholas@google.com Fixed nondeterminism in SkSL dehydration
2020-07-31 robertphillips@google.com Handle null GrDirectContext in DDL Fuzzer

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC borenet@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

* increase number of shards. make engine web tests same as flutter repo tests

* remove golden arguments. tests failed

* adding comments