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
Update ExternalViewEmbedder class comment. #11063
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm modulo nits.
@cbracken forgot to send the nits? |
flow/embedded_views.h
Outdated
// Facilitates embedding of platform views within the flow layer tree. | ||
// | ||
// Used on iOS and on embedded platforms that provides a system compositor | ||
// as part of the project arguments |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
End this sentence with a period.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM modulo nites, with the actual nits this time.
) (#40776) git@github.com:flutter/engine.git/compare/2289072724ff...113be24 git log 2289072..113be24 --no-merges --oneline 2019-09-18 amirh@users.noreply.github.com Update ExternalViewEmbedder class comment. (flutter/engine#11063) 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 stuartmorgan@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
) (flutter#40776) git@github.com:flutter/engine.git/compare/2289072724ff...113be24 git log 2289072..113be24 --no-merges --oneline 2019-09-18 amirh@users.noreply.github.com Update ExternalViewEmbedder class comment. (flutter/engine#11063) 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 stuartmorgan@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
No description provided.