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

Conversation

@gaaclarke
Copy link
Member

related issue: flutter/flutter#55778

@auto-assign auto-assign bot requested a review from iskakaushik April 28, 2020 00:03
@gaaclarke gaaclarke requested review from xster and removed request for iskakaushik April 28, 2020 00:03
@gaaclarke
Copy link
Member Author

@xster I made those changes we discussed over VC.

FLUTTER_ASSERT_ARC

@interface XCUIElement (ftr_waitForNonExistence)
/// Keeps waiting until the element doesn't exist. Returns NO if the timeout is
Copy link
Member

Choose a reason for hiding this comment

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

keep waiting until the element no longe exists?

Copy link
Member Author

Choose a reason for hiding this comment

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

That sounds like it has to exist beforehand. The implementation is agnostic to that.

@xster
Copy link
Member

xster commented Apr 28, 2020

LGTM

@gaaclarke
Copy link
Member Author

Landing before Windows Web Engine finishes, it passed previously. I just removed some print statements since then.

@gaaclarke gaaclarke merged commit 103c9c7 into flutter:master Apr 28, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 28, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 28, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 28, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants