Skip to content

Respect CHROME_EXECUTABLE in web benchmarks#49911

Merged
yjbanov merged 1 commit intoflutter:masterfrom
yjbanov:chrome-executable
Feb 1, 2020
Merged

Respect CHROME_EXECUTABLE in web benchmarks#49911
yjbanov merged 1 commit intoflutter:masterfrom
yjbanov:chrome-executable

Conversation

@yjbanov
Copy link
Copy Markdown
Contributor

@yjbanov yjbanov commented Jan 31, 2020

Description

Fix Web benchmarks on HHH bots by respecting CHROME_EXECUTABLE.

Related Issues

Fixes dart-lang/sdk#40421

@yjbanov yjbanov requested a review from alexmarkov January 31, 2020 21:41
@fluttergithubbot fluttergithubbot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Jan 31, 2020
@fluttergithubbot
Copy link
Copy Markdown
Contributor

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

Copy link
Copy Markdown
Contributor

@alexmarkov alexmarkov 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
Copy link
Copy Markdown
Contributor Author

yjbanov commented Feb 1, 2020

Merging on red to unbreak HHH builder. The Cirrus failure looks like the usual flake anyway.

@yjbanov yjbanov merged commit ff7eb77 into flutter:master Feb 1, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

web_benchmarks_canvaskit cannot find Chrome on Flutter HHH bots

4 participants