-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
c: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterp: webviewThe WebView pluginThe WebView pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.platform-androidAndroid applications specificallyAndroid applications specifically
Description
There are portions in FlutterWebView.java which have todo comments that can now be fixed :
For eg :
https://github.com/flutter/plugins/blob/fe31e5292f14eee3b7a9c3a7375bb04acd159057/packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/FlutterWebView.java#L208
Since the required code has now been shipped to stable
https://github.com/flutter/engine/blob/b3af521a050e6ef076778bcaf16e27b2521df8f8/shell/platform/android/io/flutter/plugin/platform/VirtualDisplayController.java#L192
Metadata
Metadata
Assignees
Labels
c: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterp: webviewThe WebView pluginThe WebView pluginpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.platform-androidAndroid applications specificallyAndroid applications specifically