This reverts commit e0c63cd3.

This reverts commit 7d2620be.

TBR: @flar @iskakaushik 

Fixes the broken devicelab tests.

This reverts commit d35671c6.

* Adds tilePadding property to ExpansionTile

d3f1c08f5 Roll src/third_party/skia b920a0b91b1c..5e1a57f42120 (5 commits) (flutter/engine#17880) (#55418)

* extract engine sub-metrics; change reported metrics

- Extract sub-metrics reported by the Web engine: "preroll_frame", "apply_frame".
- Add a concept of unreported metrics: displayed on the benchmark UI, but not on the dashboard.
- Make "sceneBuildDuration" and "windowRenderDuration" unreported, which are too fine-grained. They are included in "drawFrameDuration" already.
- Report outlier ratio instead of outlier average. The ratio is more useful of the two.

* Fix FlutterError.onError in debug mode

* update

* fix comments

* add license header

* fix analyzer

* update

* another attempt

* fix test

* fix comment

Revert "[flutter_tools] default tree-shake-icons to enabled and improve performance (#54923)" (#55413)

This reverts commit 0b93a921.

remove intellij references to the v1 embedding jars now that the v2 embeddings are referenced via maven (#55353)

The arrow key and delete handling in RenderEditable was using single
index values, which made it possible to move the cursor into the middle
of a surrogate pair (allowing things like adding text at that insertion
point), or to delete half of a surrogate pair. Since unpaired surrogate
pairs aren't valid UTF-16, doing so would cause assertions in the text
field.

This makes the arrow key and delete key handling surrogate-aware
(although not grapheme-cluster-aware, which is a larger fix that is out
of scope here).

Part of #55014

ef05a18c0 Roll src/third_party/dart 87b829bacd36..e6baa97e9adc (5 commits) (flutter/engine#17870) (#55368)

4616931ea Roll src/third_party/skia e9663db508df..a6cd5588d231 (4 commits) (flutter/engine#17868) (#55363)