这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Revert "[web] Remove the JS API for url strategy" #42468

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

jonahwilliams
Copy link
Contributor

Reverts #42134

This is blocking the engine into framework roller:

See: https://cirrus-ci.com/task/5610586755563520

Analyzing 3 items...                                            
  error • The class 'UrlStrategy' can't be extended outside of its library because it's an interface class • dev/integration_tests/web_e2e_tests/test_driver/url_strategy_integration.dart:48:31 • invalid_use_of_type_outside_library
1 issue found. (ran in 321.8s)
  🙙  🙛  

@flutter-dashboard flutter-dashboard bot added the platform-web Code specifically for the web engine label Jun 1, 2023
@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 1, 2023
@auto-submit auto-submit bot merged commit 6db2f3e into main Jun 1, 2023
@auto-submit auto-submit bot deleted the revert-42134-remove_js_url_strategy branch June 1, 2023 01:53
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 1, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jun 1, 2023
…128009)

flutter/engine@f9bc342...6db2f3e

2023-06-01 jonahwilliams@google.com Revert "[web] Remove the JS API for url strategy" (flutter/engine#42468)
2023-06-01 skia-flutter-autoroll@skia.org Roll Dart SDK from e0caea366989 to 78d09b5874fa (4 revisions) (flutter/engine#42465)
2023-06-01 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from lSKDoGVypQfTMYUZe... to htio0wC3kDb9tB1Wd... (flutter/engine#42463)
2023-05-31 mdebbar@google.com [web] Remove the JS API for url strategy (flutter/engine#42134)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from lSKDoGVypQfT to htio0wC3kDb9

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 jonahwilliams@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

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/+doc/main/autoroll/README.md
auto-submit bot pushed a commit that referenced this pull request Jun 1, 2023
Initially landed in #42134
Then reverted in #42468

It failed because we changed `UrlStrategy` to an `interface` which prevents "extending".

The only change in the reland is the removal of the `interface` keyword.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants