Skip to content

Conversation

renyou
Copy link
Contributor

@renyou renyou commented Apr 22, 2021

Reverts #80772

This breaks some Google internal tests. b/186079559

@renyou renyou requested review from HansMuller and xu-baolin April 22, 2021 17:02
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

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

@flutter-dashboard flutter-dashboard bot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Apr 22, 2021
@google-cla google-cla bot added the cla: yes label Apr 22, 2021
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@xu-baolin xu-baolin left a comment

Choose a reason for hiding this comment

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

LGTM

@chunhtai
Copy link
Contributor

it looks like one of our internal test is not written correctly, this may take some time to resolve since this is low priority, and I am busy with other high priority issue

@chunhtai chunhtai merged commit 7750482 into master Apr 22, 2021
@chunhtai chunhtai deleted the revert-80772-0420dynamic branch April 22, 2021 17:19
renyou added a commit to renyou/flutter that referenced this pull request Apr 22, 2021
renyou added a commit that referenced this pull request Apr 22, 2021
…80986)

* Revert "Replace some `dynamic` to `Object?` type (#80772)" (#80965)

This reverts commit 12a2e68.

* Add frontend_server_client to dependency allowlist (#80912)

Co-authored-by: Jenn Magder <magder@google.com>
renyou added a commit that referenced this pull request Apr 26, 2021
…81076 (#81155)

* Revert "Replace some `dynamic` to `Object?` type (#80772)" (#80965)

This reverts commit 12a2e68.

* Add frontend_server_client to dependency allowlist (#80912)

* Revert "[RenderEditable] Dont paint caret when selection is invalid (#79607)" (#81076)

This reverts commit 0f8148e.

* Convert AnimatedSize to a StatefulWidget (#80554)

Co-authored-by: Jenn Magder <magder@google.com>
Co-authored-by: Jason Simmons <jason-simmons@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants