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

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
Next Next commit
Default implementation for source of error, too
  • Loading branch information
@justinmc
justinmc committed Jun 12, 2023
commit d701c984998731877350f9941b7d3be51b16cc63
2 changes: 1 addition & 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 @@ -517,7 +517,7 @@ public interface PlatformMessageHandler {
void setSystemUiOverlayStyle(@NonNull SystemChromeStyle systemUiOverlayStyle);

/** The Flutter application would or would not like to handle navigation pop events itself. */
void setFrameworkHandlesBack(boolean frameworkHandlesBack);
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.


/**
* The Flutter application would like to pop the top item off of the Android app's navigation
Expand Down