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

[web] edge launcher for windows #15690

Merged
merged 6 commits into from
Jan 16, 2020
Merged

Conversation

nturgut
Copy link
Contributor

@nturgut nturgut commented Jan 16, 2020

In order to launch edge on windows from command line or from a batch script we are using the following github project:

https://github.com/MicrosoftEdge/edge-launcher

Fixes: flutter/flutter#48823

@nturgut nturgut added the platform-web Code specifically for the web engine label Jan 16, 2020
@nturgut nturgut requested a review from yjbanov January 16, 2020 17:18
@auto-assign auto-assign bot requested a review from cbracken January 16, 2020 17:18
@nturgut nturgut removed the request for review from cbracken January 16, 2020 17:18
lib/web_ui/dev/edge_installation.dart Outdated Show resolved Hide resolved
lib/web_ui/dev/edge_installation.dart Outdated Show resolved Hide resolved
lib/web_ui/dev/edge_installation.dart Outdated Show resolved Hide resolved
lib/web_ui/dev/edge_installation.dart Outdated Show resolved Hide resolved
lib/web_ui/dev/edge_installation.dart Outdated Show resolved Hide resolved
@nturgut nturgut requested a review from yjbanov January 16, 2020 18:32
@nturgut
Copy link
Contributor Author

nturgut commented Jan 16, 2020

I'll continue with the merge since only LUCI tests have an exception and the PR is only about web tooling for windows. Hopefully the run triggered with this PR will make the tree green.

@nturgut nturgut merged commit 76b36e0 into flutter:master Jan 16, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 17, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 17, 2020
flutter/engine@839b34c...0427416

git log 839b34c..0427416 --first-parent --oneline
2020-01-16 dworsham@google.com Fix Opacity performance regression on Fuchsia (flutter/engine#15573)
2020-01-16 nurhan@google.com [web] edge launcher for windows (flutter/engine#15690)
2020-01-16 jason-simmons@users.noreply.github.com [web] Add BoxHeightStyle.strut (flutter/engine#15694)


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 jimgraham@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
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
* edge launcher for windows

* addressing PR comments

* url mistake

* remove uncessary import

* initialization error fix

* format
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
* edge launcher for windows

* addressing PR comments

* url mistake

* remove uncessary import

* initialization error fix

* format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web] Install necessary tools to launch Edge
3 participants