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

Remove unused Imports and private method #15872

Merged
merged 2 commits into from
Jan 22, 2020

Conversation

goderbauer
Copy link
Member

These are unused as reported during a google3 roll.

@goderbauer goderbauer changed the title Remove Imports Remove unused Imports and private method Jan 22, 2020
Copy link
Contributor

@matthew-carroll matthew-carroll left a comment

Choose a reason for hiding this comment

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

LGTM

@goderbauer goderbauer added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Jan 22, 2020
@fluttergithubbot fluttergithubbot merged commit 0615f45 into flutter:master Jan 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 22, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 23, 2020
flutter/engine@83a64b7...46e58b9

git log 83a64b7..46e58b9 --first-parent --oneline
2020-01-22 iska.kaushik@gmail.com [bots] remove auto-assign for web PRs (flutter/engine#15878)
2020-01-22 dnfield@google.com [fuchsia] HitTesting for fuchsia a11y (flutter/engine#15570)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/skia 808f021b51df..d59053987a27 (4 commits) (flutter/engine#15882)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/dart 4ad5fab95753..439cabaec02f (2 commits) (flutter/engine#15881)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/skia 4277f0173657..808f021b51df (7 commits) (flutter/engine#15879)
2020-01-22 goderbauer@google.com Remove unused Imports and private method (flutter/engine#15872)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/dart ef0c7f16e609..4ad5fab95753 (41 commits) (flutter/engine#15875)
2020-01-22 stuartmorgan@google.com Add missing include in GLFW and Windows embeddings (flutter/engine#15867)
2020-01-22 skia-flutter-autoroll@skia.org Roll fuchsia/sdk/core/linux-amd64 from p1UDn... to v-OJE... (flutter/engine#15874)


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

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/+/master/autoroll/README.md
jonahwilliams pushed a commit to jonahwilliams/flutter that referenced this pull request Jan 24, 2020
flutter/engine@83a64b7...46e58b9

git log 83a64b7..46e58b9 --first-parent --oneline
2020-01-22 iska.kaushik@gmail.com [bots] remove auto-assign for web PRs (flutter/engine#15878)
2020-01-22 dnfield@google.com [fuchsia] HitTesting for fuchsia a11y (flutter/engine#15570)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/skia 808f021b51df..d59053987a27 (4 commits) (flutter/engine#15882)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/dart 4ad5fab95753..439cabaec02f (2 commits) (flutter/engine#15881)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/skia 4277f0173657..808f021b51df (7 commits) (flutter/engine#15879)
2020-01-22 goderbauer@google.com Remove unused Imports and private method (flutter/engine#15872)
2020-01-22 skia-flutter-autoroll@skia.org Roll src/third_party/dart ef0c7f16e609..4ad5fab95753 (41 commits) (flutter/engine#15875)
2020-01-22 stuartmorgan@google.com Add missing include in GLFW and Windows embeddings (flutter/engine#15867)
2020-01-22 skia-flutter-autoroll@skia.org Roll fuchsia/sdk/core/linux-amd64 from p1UDn... to v-OJE... (flutter/engine#15874)


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

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/+/master/autoroll/README.md
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
@goderbauer goderbauer deleted the removeImprots branch March 8, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.
Projects
None yet
4 participants