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

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Aug 15, 2019

This PR pulls the latest version of the Web engine and enables Web engine tests on Cirrur.

@@ -0,0 +1,89 @@
part of engine;
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing license header

Copy link
Contributor Author

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;
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing license header

Copy link
Contributor Author

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;
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing license header

Copy link
Contributor Author

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.
Copy link
Contributor

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

Copy link
Contributor Author

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;
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing license header

Copy link
Contributor Author

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.
Copy link
Contributor

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

Copy link
Contributor Author

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: |
Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Done.

@yjbanov
Copy link
Contributor Author

yjbanov commented Aug 15, 2019

@jonahwilliams Also added a license test so this doesn't happen in the future.

@dnfield
Copy link
Contributor

dnfield commented Aug 15, 2019

@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.

@yjbanov
Copy link
Contributor Author

yjbanov commented Aug 15, 2019

@dnfield How do I configure LUCI to run these tests?

@dnfield
Copy link
Contributor

dnfield commented Aug 15, 2019

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.

@yjbanov yjbanov force-pushed the web-tests branch 4 times, most recently from 95556a0 to 55c3bba Compare August 22, 2019 17:17
@yjbanov yjbanov requested a review from ferhatb August 22, 2019 20:51
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM!

@yjbanov yjbanov merged commit fad38e2 into flutter:master Aug 22, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 23, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 23, 2019
[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.
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