Skip to content

Conversation

mdebbar
Copy link
Contributor

@mdebbar mdebbar commented Sep 6, 2019

Flutter web doesn't support the performance overlay yet. This PR disables the performance overlay service extension so the tools can disable the overlay accordingly.

@fluttergithubbot fluttergithubbot added the framework flutter/packages/flutter repository. See also f: labels. label Sep 6, 2019
@fluttergithubbot
Copy link
Contributor

It looks like this pull request may not have tests. Please make sure to add tests before merging. While there are exceptions to this rule, if this patch modifies code it is probably not an exception.

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

@mdebbar mdebbar self-assigned this Sep 6, 2019
@mdebbar mdebbar added the platform-web Web applications specifically label Sep 6, 2019
Copy link
Contributor

@ferhatb ferhatb left a comment

Choose a reason for hiding this comment

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

Bindings test should be updated too ?

@mdebbar
Copy link
Contributor Author

mdebbar commented Sep 6, 2019

@ferhatb bindings tests don't check any service extensions. There's a foundation/service_extensions_test.dart that checks all service extensions. But it doesn't run on web (it has @TestOn('!chrome')). Not sure where/how to test this (cc @jonahwilliams?).

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

@mdebbar mdebbar merged commit 03e8100 into flutter:master Sep 10, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
framework flutter/packages/flutter repository. See also f: labels. platform-web Web applications specifically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants