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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Apr 22, 2019

I inadvertently removed a comma in my last PR. This adds a test to make sure that's good.

@dnfield dnfield requested a review from cbracken April 22, 2019 20:53
@dnfield
Copy link
Contributor Author

dnfield commented Apr 22, 2019

Landing on red since framework test failure are expected.

@dnfield dnfield merged commit 9fa7336 into flutter:master Apr 22, 2019
@dnfield dnfield deleted the const_rect_64 branch April 22, 2019 21:12
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
dnfield added a commit to dnfield/engine that referenced this pull request Apr 22, 2019
dnfield added a commit that referenced this pull request Apr 22, 2019
* Revert "fix toString (#8688)"

This reverts commit 9fa7336.

* Revert "Make Rect and RRect use 64 bit doubles, and make them const-able (#8565)"

This reverts commit c123152.
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.
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.

3 participants