Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

fuchsia: Remove dead code / break dependencies #19396

Merged
merged 9 commits into from
Aug 4, 2020

Conversation

arbreng
Copy link
Contributor

@arbreng arbreng commented Jun 29, 2020

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

Copy link
Contributor

@gw280 gw280 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I'm not an expert on the layers code. I'd like @chinmaygarde or someone else to also LGTM this before it's merged.

@arbreng arbreng force-pushed the fuchsia-dead-code branch 2 times, most recently from 502d833 to 3587761 Compare June 30, 2020 20:32
@arbreng arbreng force-pushed the fuchsia-dead-code branch 2 times, most recently from 7cf66c1 to 6fa5c32 Compare July 3, 2020 21:45
@arbreng arbreng force-pushed the fuchsia-dead-code branch from 6fa5c32 to 1e8c3c1 Compare July 14, 2020 00:38
@arbreng arbreng force-pushed the fuchsia-dead-code branch 2 times, most recently from 7e2b52d to 2348802 Compare July 21, 2020 20:26
@arbreng arbreng requested a review from freiling July 21, 2020 20:30
@arbreng arbreng force-pushed the fuchsia-dead-code branch 3 times, most recently from 6105238 to 8e0750d Compare July 22, 2020 21:30
@chinmaygarde
Copy link
Member

@arbreng Can we land or close this?

@arbreng arbreng force-pushed the fuchsia-dead-code branch from 8e0750d to 7134e10 Compare July 30, 2020 23:08
@arbreng
Copy link
Contributor Author

arbreng commented Jul 31, 2020

@arbreng Can we land or close this?

Yep, trying to do so today :) There were several internal tests failing on this CL, so I wanted to make sure they worked first. Now to get it past Flutter's CI..

@arbreng arbreng force-pushed the fuchsia-dead-code branch from a9d6cc0 to 8717760 Compare August 4, 2020 03:03
@arbreng arbreng force-pushed the fuchsia-dead-code branch from 8717760 to aaa1e00 Compare August 4, 2020 03:33
@arbreng arbreng merged commit 12a3747 into flutter:master Aug 4, 2020
@arbreng arbreng deleted the fuchsia-dead-code branch August 4, 2020 08:25
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 4, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 4, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 4, 2020
zanderso added a commit that referenced this pull request Aug 6, 2020
zanderso added a commit that referenced this pull request Aug 6, 2020
renyou pushed a commit to renyou/engine that referenced this pull request Aug 7, 2020
renyou added a commit that referenced this pull request Aug 7, 2020
arbreng added a commit that referenced this pull request Aug 15, 2020
Reland #19396 with a fix for improper scale that was affecting internal tests

Tested: Ran all unittests, ran internal tests, and ran workstation on Fuchsia
BUG: 53062, 53063
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants