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

Delete code to fallback to the dart:html version of the app. #2713

Merged
merged 1 commit into from
Feb 18, 2021

Conversation

jacob314
Copy link
Contributor

Leave the index_fallback.html as an escape hatch to debug if users hit issues or IDEs need to fallback.

Leave the index_fallback.html as an escape hatch to debug if users hit issues or IDEs need to fallback.
@jacob314
Copy link
Contributor Author

Fixes #2598 unless we are really crashing canvaskit.

import 'src/debugger/syntax_highlighter.dart';
import 'src/preferences.dart';

void main() async {
final defaultOnError = FlutterError.onError;
Copy link
Member

@kenzieschmoll kenzieschmoll Feb 18, 2021

Choose a reason for hiding this comment

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

Can you please also submit a CL with this change to the internal main.dart file? That file is not updated as part of the roll.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good news is as the files it depends on were deleted you should get a compile error as part of the next roll. I can do the next roll into internal and fix as part of that.

@jacob314 jacob314 merged commit 5ec3df5 into flutter:master Feb 18, 2021
@jacob314 jacob314 deleted the trust_canvaskit branch February 18, 2021 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants