This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Merged
Conversation
PR flutter#8048 in flutter/engine produced a bug (flutter/flutter flutter#34791) in flutter for Android that doesn't permit the Recents app bar color to be changed. This restores the original arguments to the function found in the previous version (https://github.com/flutter/engine/blob/2f4a38dbd33061e950cf83ddc31785803d58c126/shell/platform/android/io/flutter/plugin/platform/PlatformPlugin.java) of PlatformPlugin.java while preserving the enhancements for the linter.
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
1 similar comment
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
Contributor
Author
|
I signed it! |
|
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
1 similar comment
|
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
mklim
approved these changes
Aug 5, 2019
shell/platform/android/io/flutter/plugin/platform/PlatformPlugin.java
Outdated
Show resolved
Hide resolved
…in.java Annotate the icon parameter for anybody reading the code who doesn't know the whole method signature. Co-Authored-By: Michael Klimushyn <[email protected]>
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Aug 15, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Aug 15, 2019
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Aug 16, 2019
engine-flutter-autoroll
added a commit
to flutter/flutter
that referenced
this pull request
Aug 16, 2019
[email protected]:flutter/engine.git/compare/f8e7453f1106...4b7a552 git log f8e7453..4b7a552 --no-merges --oneline 2019-08-16 [email protected] Roll Dart back to e35e8833 (flutter/engine#11043) 2019-08-16 [email protected] Roll src/third_party/dart 306f8e04bb..fecc4c8f2d (4 commits) 2019-08-16 [email protected] Roll src/third_party/skia 963a606677e1..6a519b8dd895 (6 commits) (flutter/engine#11042) 2019-08-15 [email protected] Hide verbose dart snapshot during run_test.py (flutter/engine#11040) 2019-08-15 [email protected] Remove ability to override mac_sdk_path in flutter/tools/gn (flutter/engine#11013) 2019-08-15 [email protected] Roll src/third_party/dart cd16fba718..306f8e04bb (10 commits) 2019-08-15 [email protected] Roll buildroot to pick up recent macOS changes (flutter/engine#11037) 2019-08-15 [email protected] Remove the ParagraphImpl class from the text API (flutter/engine#11012) 2019-08-15 [email protected] Disable a deprecation warning for use of a TaskDescription constructor for older platforms (flutter/engine#11029) 2019-08-15 [email protected] Re-lands platform brightness support on iOS, plus platform contrast (flutter/engine#10791) 2019-08-15 [email protected] [fuchsia] Add required trace so files for fuchsia fars (flutter/engine#11036) 2019-08-15 [email protected] Roll src/third_party/skia e5dc1ebc864a..963a606677e1 (14 commits) (flutter/engine#11032) 2019-08-15 [email protected] Add _glfw versions of the GLFW desktop libraries (flutter/engine#11024) 2019-08-15 [email protected] Roll fuchsia/sdk/core/linux-amd64 from _fvZN... to 5Nhwb... (flutter/engine#11028) 2019-08-15 [email protected] Roll src/third_party/dart 9552646dc4..cd16fba718 (5 commits) 2019-08-15 [email protected] Fix first frame logic (flutter/engine#11027) 2019-08-15 [email protected] remove OS version (flutter/engine#11033) 2019-08-15 [email protected] Roll src/third_party/skia e30a485a68c9..e5dc1ebc864a (7 commits) (flutter/engine#11025) 2019-08-15 [email protected] Roll src/third_party/dart cae08c6813..9552646dc4 (3 commits) 2019-08-15 [email protected] Remove the output directory prefix from the Android engine JAR filename (flutter/engine#11015) 2019-08-15 [email protected] Fix flutter/flutter #34791 (flutter/engine#9977) 2019-08-15 [email protected] Roll src/third_party/dart e35e8833ee..cae08c6813 (28 commits) 2019-08-15 [email protected] Roll src/third_party/skia f3f50099533d..e30a485a68c9 (2 commits) (flutter/engine#11022) 2019-08-15 [email protected] Roll src/third_party/skia 319fd3d7bcb4..f3f50099533d (4 commits) (flutter/engine#11021) The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff ([email protected]), and stop the roller if necessary.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a fix for flutter/flutter issue #34791.
PR #8048 in flutter/engine produced a bug/regression (flutter/flutter #34791) in flutter for Android that doesn't permit the Recents app bar color to be changed. This restores the original arguments to the function found in the previous version (https://github.com/flutter/engine/blob/2f4a38dbd33061e950cf83ddc31785803d58c126/shell/platform/android/io/flutter/plugin/platform/PlatformPlugin.java) of PlatformPlugin.java while preserving the enhancements for the linter.
I've compiled and tested this fix locally. The bar changes color again.