Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Predictive back breakage fix #42789

Merged
justinmc merged 6 commits into from
Jun 12, 2023
Merged

Predictive back breakage fix #42789

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
0734cdf
Provide a default implementation to avoid needing to override setFram…
justinmc Jun 12, 2023
782eee5
Docs to explain the empty method
justinmc Jun 12, 2023
cb003af
Formatting
justinmc Jun 12, 2023
41b893d
frameworkHandlesBacks => frameworkHandlesBack
justinmc Jun 12, 2023
d701c98
Default implementation for source of error, too
justinmc Jun 12, 2023
ab5c5cd
Clarify docs
justinmc Jun 12, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Clarify docs
  • Loading branch information
@justinmc
justinmc committed Jun 12, 2023
commit ab5c5cd482d123ab24312cf9463b3ed6dca188bc
8 changes: 7 additions & 1 deletion shell/platform/android/io/flutter/embedding/engine/systemchannels/PlatformChannel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,13 @@ public interface PlatformMessageHandler {
*/
void setSystemUiOverlayStyle(@NonNull SystemChromeStyle systemUiOverlayStyle);

/** The Flutter application would or would not like to handle navigation pop events itself. */
/**
* The Flutter application would or would not like to handle navigation pop events itself.
*
* <p>Relevant for registering and unregistering the app's OnBackInvokedCallback for the
* Predictive Back feature, for example as in {@link
* io.flutter.embedding.android.FlutterActivity}.
*/
default void setFrameworkHandlesBack(boolean frameworkHandlesBack) {}
Copy link
Contributor

@camsim99 camsim99 Jun 12, 2023

Choose a reason for hiding this comment

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

This makes sense to me since it is only used in a particular place. I'd suggest expanding the doc to mention where exactly this is relevant currently (FlutterActivity), just so it's clear we don't expect it to work or be used elsewhere.


/**
Expand Down
3 changes: 2 additions & 1 deletion shell/platform/android/io/flutter/plugin/platform/PlatformPlugin.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ public interface PlatformPluginDelegate {
* The Flutter application would or would not like to handle navigation pop events itself.
*
* <p>Relevant for registering and unregistering the app's OnBackInvokedCallback for the
* Predictive Back feature.
* Predictive Back feature, for example as in {@link
* io.flutter.embedding.android.FlutterActivity}.
*/
default void setFrameworkHandlesBack(boolean frameworkHandlesBack) {}
Copy link
Contributor

@camsim99 camsim99 Jun 12, 2023

Choose a reason for hiding this comment

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

I think this works as a good alternative to the empty implementation in FlutterFragment. Maybe the doc can expand on where this is currently relevant (FlutterActivity alone now I suppose)?

}
Expand Down