-
Notifications
You must be signed in to change notification settings - Fork 6k
sync web engine; run web engine tests #11031
Conversation
| @@ -0,0 +1,89 @@ | |||
| part of engine; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing license header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
| @@ -0,0 +1,58 @@ | |||
| part of engine; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing license header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
| @@ -0,0 +1,272 @@ | |||
| part of engine; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing license header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
| @@ -1,4 +1,4 @@ | |||
| // Copyright 2013 The Flutter Authors. All rights reserved. | |||
| // Copyright 2018 The Flutter Authors. All rights reserved. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't change license dates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
| @@ -0,0 +1,207 @@ | |||
| part of engine; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing license header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
| @@ -1,7 +1,3 @@ | |||
| // Copyright 2013 The Flutter Authors. All rights reserved. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re-add this license header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
.cirrus.yml
Outdated
| test_host_script: | | ||
| cd $ENGINE_PATH/src | ||
| ./flutter/testing/run_tests.sh host_debug_unopt | ||
| test_web_engine: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this has to be postfixed _script, so test_web_engine_script.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Done.
|
@jonahwilliams Also added a license test so this doesn't happen in the future. |
|
@yjbanov - I would recommend investing time in gettin gthese tests running on LUCI rather than on Cirrus at this point, if they're not already running on LUCI. |
|
@dnfield How do I configure LUCI to run these tests? |
|
https://github.com/flutter/flutter/tree/master/dev/bots has some docs that might help, but they're probably partly out of date at this point. We can sync offline too on this. |
95556a0 to
55c3bba
Compare
jonahwilliams
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
[email protected]:flutter/engine.git/compare/78ac65320bde...975a8aa git log 78ac653..975a8aa --no-merges --oneline 2019-08-23 [email protected] Roll src/third_party/dart 022585cfe2..06d3769ac6 (12 commits) 2019-08-23 [email protected] Roll src/third_party/skia 941d0a365045..f17d1c952987 (1 commits) (flutter/engine#11393) 2019-08-23 [email protected] Wire up software rendering in the test compositor. (flutter/engine#11392) 2019-08-23 [email protected] Roll src/third_party/dart 88c0c11612..022585cfe2 (14 commits) 2019-08-23 [email protected] preventDefault on touchend to show iOS keyboard (flutter/engine#11390) 2019-08-23 [email protected] Roll src/third_party/skia 2a5954140b49..941d0a365045 (6 commits) (flutter/engine#11389) 2019-08-22 [email protected] Allow overriding the GLFW pixel ratio (flutter/engine#11388) 2019-08-22 [email protected] Allow non-resizable windows in GLFW embedding (flutter/engine#11386) 2019-08-22 [email protected] sync web engine; run web engine tests (flutter/engine#11031) 2019-08-22 [email protected] Reland "Track detailed LibTxt metrics with LineMetrics(#10127)" (flutter/engine#11064) 2019-08-22 [email protected] Remove empty line in the pom file (flutter/engine#11382) 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.
This PR pulls the latest version of the Web engine and enables Web engine tests on Cirrur.