Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open source canvas tests from flutter_web_ui #12819

Merged
merged 3 commits into from Oct 8, 2019
Merged

Open source canvas tests from flutter_web_ui #12819

merged 3 commits into from Oct 8, 2019

Conversation

ditman
Copy link
Member

@ditman ditman commented Oct 5, 2019

Reimplement some painter tests as canvas screenshot tests in the engine.

Needs flutter/goldens#53
Fixes flutter/flutter#42027

(Set to WIP because I need to get the goldens pushed, then update the hash here)

@ditman ditman added the Work in progress (WIP) Not ready (yet) for review! label Oct 5, 2019
@ditman
Copy link
Member Author

ditman commented Oct 5, 2019

ERROR: Some files have trailing spaces. To fix, try something like `find . -name *.dart -exec sed -i -e 's/\s\+$//' {} \;`.

🤦‍♂

Reimplement some painter tests as screenshot tests on canvas.

Needs flutter/goldens#53
Fixes flutter/flutter#42027
* Add latest hash to dev/goldens_lock.yaml
* Rename (with git mv) *scuba_test.dart -> *golden_test.dart
@ditman
Copy link
Member Author

ditman commented Oct 7, 2019

Force-pushed after rebasing the branch

@ditman ditman removed the Work in progress (WIP) Not ready (yet) for review! label Oct 7, 2019
@ditman ditman merged commit f48f0fa into flutter:master Oct 8, 2019
@ditman ditman deleted the issues/42027 branch October 8, 2019 16:40
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 8, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Oct 8, 2019
git@github.com:flutter/engine.git/compare/49dbaae818cd...3a445ed

git log 49dbaae..3a445ed --no-merges --oneline
2019-10-08 30870216+gaaclarke@users.noreply.github.com Made _printDebug only happen on debug builds of the engine for now. (flutter/engine#12980)
2019-10-08 devoncarew@google.com remove references to package:_chrome (flutter/engine#12931)
2019-10-08 ditman@gmail.com Open source canvas tests from flutter_web_ui (flutter/engine#12819)
2019-10-08 mouad.debbar@gmail.com Prevent default when Tab is clicked (flutter/engine#12986)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@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/+/master/autoroll/README.md
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
git@github.com:flutter/engine.git/compare/49dbaae818cd...3a445ed

git log 49dbaae..3a445ed --no-merges --oneline
2019-10-08 30870216+gaaclarke@users.noreply.github.com Made _printDebug only happen on debug builds of the engine for now. (flutter/engine#12980)
2019-10-08 devoncarew@google.com remove references to package:_chrome (flutter/engine#12931)
2019-10-08 ditman@gmail.com Open source canvas tests from flutter_web_ui (flutter/engine#12819)
2019-10-08 mouad.debbar@gmail.com Prevent default when Tab is clicked (flutter/engine#12986)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@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/+/master/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants