-
Notifications
You must be signed in to change notification settings - Fork 19
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 device_profile #131
Remove device_profile #131
Conversation
4d83d96
to
aaf315e
Compare
Have you test the behavior on each device? or Do you need help testing on each device? |
@bbrto21 No, I haven't. I'm out of the office until tomorrow. I'll be grateful if you run the test for me (if you think this change is reasonable). |
I've test this on wearable, tv, mobile device. it works properly! |
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.
Others look good to me
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!
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
* Remove device_profile * Remove flutter:: from key_event_channel.cc * Remove system_info dependency
It's unnecessary to read the current device profile at runtime.
The use of preprocessor macro is discouraged by the Google style guide, so it's
rather than