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

Conversation

@chinmaygarde
Copy link
Member

This reverts commit 4ead9c8.

Reland tracked in flutter/flutter#31433.

Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

LGTM

@chinmaygarde
Copy link
Member Author

Failing presubmit addressed by pending manual roll.

@chinmaygarde chinmaygarde merged commit 6dc5dca into flutter:master Apr 22, 2019
@chinmaygarde chinmaygarde deleted the revert_settings_to_string branch April 22, 2019 21:31
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 23, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 23, 2019
flutter/engine@934772d...0523870

git log 934772d..0523870 --no-merges --oneline
0523870 Add tests from framework (flutter/engine#8692)
ed1f3fd Change Vertices.indices to use a Uint16 list to more accurately reflect Skia's API (flutter/engine#8657)
4f2fd84 Revert Rect/RRect 64 bit (flutter/engine#8690)
6dc5dca Revert "Remove unused Settings::ToString. (#8642)" (flutter/engine#8689)
9fa7336 fix toString (flutter/engine#8688)
c123152 Make Rect and RRect use 64 bit doubles, and make them const-able (flutter/engine#8565)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

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

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
@eggfly
Copy link
Member

eggfly commented Mar 15, 2022

I also found this ToString function. Why can't we delete it?

@zanderso
Copy link
Member

Since the Fuchsia runner is in-tree, I'm guessing the ToString can now be deleted more easily.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants