[webview_flutter] add mixedContentMode option for Android WebView

[dotcink]

Description

Add support of mixedContentMode for Android WebView.

Related Issues

Fixes flutter/flutter#43595 and flutter/flutter#24975

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
• All existing and new tests are passing.
• I updated/added relevant documentation (doc comments with ///).
• The analyzer (flutter analyze) does not report any problems on my PR.
• I read and followed the Flutter Style Guide.
• The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
• I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
• I updated CHANGELOG.md to add a description of the change.
• I signed the CLA.
• I am willing to follow-up on review comments in a timely manner.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

• Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
• No, this is not a breaking change.

[dotcink]

@amirh Please Take A Look.
Thank you.

[dotcink]

Not sure what's the meaning of branch-guard — Cirrus CI failure.
This failing check didn't show at first.
Clicking Details shows error message:

{
  "message": "If you would like to help us test the Checks API during its preview period, you must specify a custom media type in the 'Accept' header. Please see the docs for full details.",
  "documentation_url": "https://developer.github.com/v3/checks/suites/#get-a-single-check-suite"
}

[edisonywh]

Thanks for the great work @dotcink! Any update on this, @amirh? Right now my app just looks pretty jarring with unloaded images everywhere.

[dotcink]

Thanks for the great work @dotcink! Any update on this, @amirh? Right now my app just looks pretty jarring with unloaded images everywhere.

Please thumbs up 👍 flutter/flutter#43595, Flutter team use this to guide the issue prioritizing.

[ikkyu0319]

nice ~

[Vasques1995]

Hi @dotcink any news on the issue this PR is trying to address? The only reason that the PR is not merged is because of the failed checks?

[dotcink]

Hi @dotcink any news on the issue this PR is trying to address? The only reason that the PR is not merged is because of the failed checks?

I just submitted another common solution. This PR might close if possible.

[stuartmorgan-g]

Thanks for the submission! We’re currently working through a large backlog of PRs, and this will require non-trivial review, so it will take some time before we’re able to review it. As explained in CONTRIBUTING.md, votes for the corresponding issue are the primary way we’re prioritizing non-trivial reviews, so we encourage anyone interested in this PR to vote for the corresponding issue.

[ditman]

I just submitted another common solution. This PR might close if possible.

Closing in favor of: #3721

[dotcink]

I just submitted another common solution. This PR might close if possible.

Closing in favor of: #3721

So would #3721 be prepared to be reviewed and merged please?

[stuartmorgan-g]

I just submitted another common solution. This PR might close if possible.

Closing in favor of: #3721

There isn't a decision yet that that's the way we want to go, so for now both should stay open.

[stuartmorgan-g]

So would #3721 be prepared to be reviewed and merged please?

Please see #3721 (comment)

[stuartmorgan-g]

Adding a setting for this makes sense in the new structure we'll be using, but it's blocked on flutter/flutter#94051: we'll want to support platform extensions to WebSettings instead of adding this to the main class, and have a way of configuring the web view that doesn't require yet another highly-specific optional parameter.

We can leave this open for now and it can be adjusted to fit the new structure once that work is done.

(/cc @bparrishMines and @mvanbeusekom who will be working on that restructing.)

[stuartmorgan-g]

Status update from triage: Still blocked on flutter/flutter#94051, which is being worked on.

[Hixie]

Latest status is that we're still waiting on flutter/flutter#94051.
Before this can land even after that is fixed, the conflicts will need to be resolved.

[dotcink]

Latest status is that we're still waiting on flutter/flutter#94051. Before this can land even after that is fixed, the conflicts will need to be resolved.

OK, I'll resolve the conflicts according to the new solution.

[stuartmorgan-g]

Status update from triage: still blocked, but we hope to be unblocked soon.

[stuartmorgan-g]

webview_flutter v4 has finally been completed, which included fixing the platform interface to that we can make additions to it without breaking changes. That means that this is finally unblocked! Thanks for your patience while we worked through the restructuring.

webview_flutter v4 uses a Dart-first implementation, where the platform channel boundary is as close as possible to the underlying host APIs, and plugin logic is written primarily in Dart rather than the host language. That means that this PR will need to be updated to follow that model. Please let us know if you have questions about updating to v4 (and #6881 may be useful as an example of adding a feature in the v4 structure).

[stuartmorgan-g]

@dotcink Are you still planning on updating this for compatibility with v4?

[stuartmorgan-g]

(Marking this as a Draft pending that update.)

[dotcink]

OK, I'll check the new v4 in the weekend.

[stuartmorgan-g]

We've just completed the migration of the plugin code to the flutter/packages repository, as described in https://flutter.dev/go/flutter-plugins-repo-migration, and this repository is now being archived. Unfortunately that means that all in-progress PRs here must be moved to flutter/packages.

Please see our instructions for an explanation of how to move your PR, and if you have any issues moving your PR please don't hesitate to reach out in the #hackers-ecosystem channel in Discord.

Our apologies that your PR was caught in this one-time transition. We're aware that it's disruptive in the short term, and appreciate your help in getting us to a better long-term state!