Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Remove usage of deprecated Fuchsia event source#37641

Merged
akbiggs merged 1 commit into
flutter-team-archive:mainfrom
crjohns:patch-1
Nov 15, 2022
Merged

Remove usage of deprecated Fuchsia event source#37641
akbiggs merged 1 commit into
flutter-team-archive:mainfrom
crjohns:patch-1

Conversation

@crjohns

@crjohns crjohns commented Nov 15, 2022

Copy link
Copy Markdown
Contributor

This capability is being removed by the Fuchsia platform (https://fuchsia-review.git.corp.google.com/c/fuchsia/+/758606), but is currently blocked on this usage.

This change simply removes the capability from this tests, where it doesn't seem to be used at all.

Fixes flutter/flutter#115392

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • [n/a] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

This capability is being removed by the Fuchsia platform (https://fuchsia-review.git.corp.google.com/c/fuchsia/+/758606), but is currently blocked on this usage.

This change simply removes the capability from this tests, where it doesn't seem to be used at all.

@lin-erik lin-erik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

EventSource was used by the fuchsia.git version of the text-input integration test for Chromium test cases so we should be good here

@gbbosak

gbbosak commented Nov 15, 2022

Copy link
Copy Markdown

It looks like Mac tests are failing, can someone on flutter re-queue the run?

@lin-erik

Copy link
Copy Markdown
Contributor

It looks like Mac tests are failing, can someone on flutter re-queue the run?

Rerunning the Mac tests

@akbiggs akbiggs merged commit b74c2c5 into flutter-team-archive:main Nov 15, 2022
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Nov 16, 2022
* 6563c58 Remove felt snapshotting behavior. (flutter-team-archive/engine#37639)

* c6e5564 Update language version in flutter_frontend_server/test/fixtures (flutter-team-archive/engine#37643)

* eedb93e Update text editing tests (flutter-team-archive/engine#37642)

* a1dd335 colorFilter as imageFilter for web (flutter-team-archive/engine#37522)

* b74c2c5 Remove usage of deprecated Fuchsia event source (flutter-team-archive/engine#37641)

* e56ed93 Revert "Update text editing tests (#37642)" (flutter-team-archive/engine#37653)
schwa423 pushed a commit to schwa423/engine that referenced this pull request Nov 16, 2022
…#37641)

This capability is being removed by the Fuchsia platform (https://fuchsia-review.git.corp.google.com/c/fuchsia/+/758606), but is currently blocked on this usage.

This change simply removes the capability from this tests, where it doesn't seem to be used at all.
shogohida pushed a commit to shogohida/flutter that referenced this pull request Dec 7, 2022
…5406)

* 6563c58 Remove felt snapshotting behavior. (flutter-team-archive/engine#37639)

* c6e5564 Update language version in flutter_frontend_server/test/fixtures (flutter-team-archive/engine#37643)

* eedb93e Update text editing tests (flutter-team-archive/engine#37642)

* a1dd335 colorFilter as imageFilter for web (flutter-team-archive/engine#37522)

* b74c2c5 Remove usage of deprecated Fuchsia event source (flutter-team-archive/engine#37641)

* e56ed93 Revert "Update text editing tests (flutter#37642)" (flutter-team-archive/engine#37653)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…5406)

* 6563c58 Remove felt snapshotting behavior. (flutter-team-archive/engine#37639)

* c6e5564 Update language version in flutter_frontend_server/test/fixtures (flutter-team-archive/engine#37643)

* eedb93e Update text editing tests (flutter-team-archive/engine#37642)

* a1dd335 colorFilter as imageFilter for web (flutter-team-archive/engine#37522)

* b74c2c5 Remove usage of deprecated Fuchsia event source (flutter-team-archive/engine#37641)

* e56ed93 Revert "Update text editing tests (flutter#37642)" (flutter-team-archive/engine#37653)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

text-input-test uses unstable Fuchsia platform features

4 participants