This repository was archived by the owner on Feb 25, 2025. It is now read-only.
sync web engine; run web engine tests #11031
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| @@ -0,0 +1,89 @@ | |||
| part of engine; | |||
Contributor
There was a problem hiding this comment.
Missing license header
| @@ -0,0 +1,58 @@ | |||
| part of engine; | |||
Contributor
There was a problem hiding this comment.
Missing license header
| @@ -0,0 +1,272 @@ | |||
| part of engine; | |||
Contributor
There was a problem hiding this comment.
Missing license header
| @@ -1,4 +1,4 @@ | |||
| // Copyright 2013 The Flutter Authors. All rights reserved. | |||
| // Copyright 2018 The Flutter Authors. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Don't change license dates
| @@ -0,0 +1,207 @@ | |||
| part of engine; | |||
Contributor
There was a problem hiding this comment.
Missing license header
| @@ -1,7 +1,3 @@ | |||
| // Copyright 2013 The Flutter Authors. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Re-add this license header
dnfield
reviewed
Aug 15, 2019
.cirrus.yml
Outdated
| test_host_script: | | ||
| cd $ENGINE_PATH/src | ||
| ./flutter/testing/run_tests.sh host_debug_unopt | ||
| test_web_engine: | |
Contributor
There was a problem hiding this comment.
this has to be postfixed _script, so test_web_engine_script.
Contributor
Author
|
@jonahwilliams Also added a license test so this doesn't happen in the future. |
Contributor
|
@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. |
Contributor
Author
|
@dnfield How do I configure LUCI to run these tests? |
Contributor
|
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
ferhatb
suggested changes
Aug 22, 2019
ferhatb
approved these changes
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
git@github.com:flutter/engine.git/compare/78ac65320bde...975a8aa git log 78ac653..975a8aa --no-merges --oneline 2019-08-23 bkonyi@google.com Roll src/third_party/dart 022585cfe2..06d3769ac6 (12 commits) 2019-08-23 skia-flutter-autoroll@skia.org Roll src/third_party/skia 941d0a365045..f17d1c952987 (1 commits) (flutter/engine#11393) 2019-08-23 chinmaygarde@google.com Wire up software rendering in the test compositor. (flutter/engine#11392) 2019-08-23 bkonyi@google.com Roll src/third_party/dart 88c0c11612..022585cfe2 (14 commits) 2019-08-23 yjbanov@google.com preventDefault on touchend to show iOS keyboard (flutter/engine#11390) 2019-08-23 skia-flutter-autoroll@skia.org Roll src/third_party/skia 2a5954140b49..941d0a365045 (6 commits) (flutter/engine#11389) 2019-08-22 stuartmorgan@google.com Allow overriding the GLFW pixel ratio (flutter/engine#11388) 2019-08-22 stuartmorgan@google.com Allow non-resizable windows in GLFW embedding (flutter/engine#11386) 2019-08-22 yjbanov@google.com sync web engine; run web engine tests (flutter/engine#11031) 2019-08-22 garyq@google.com Reland "Track detailed LibTxt metrics with LineMetrics(#10127)" (flutter/engine#11064) 2019-08-22 egarciad@google.com 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 (liyuqian@google.com), and stop the roller if necessary.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR pulls the latest version of the Web engine and enables Web engine tests on Cirrur.