Skip to content

Commit df9a0a9

Browse files
committed
Merge branch 'master' into add_zoom_to_android_webview
* master: [url_launcher] forceSafariVC should be nullable to avoid breaking change (flutter#3354) [image_picker] Do not copy a static field into another static field (flutter#3353) [video_player] Fix video player test (flutter#3361)
2 parents b45594c + 0f1c20d commit df9a0a9

File tree

10 files changed

+37
-7
lines changed

10 files changed

+37
-7
lines changed

packages/image_picker/image_picker/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 0.6.7+19
2+
3+
* Do not copy static field to another static field.
4+
15
## 0.6.7+18
26

37
* Fix outdated links across a number of markdown files ([#3276](https://github.com/flutter/plugins/pull/3276))

packages/image_picker/image_picker/lib/image_picker.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export 'package:image_picker_platform_interface/image_picker_platform_interface.
2727
class ImagePicker {
2828
/// The platform interface that drives this plugin
2929
@visibleForTesting
30-
static ImagePickerPlatform platform = ImagePickerPlatform.instance;
30+
static ImagePickerPlatform get platform => ImagePickerPlatform.instance;
3131

3232
/// Returns a [File] object pointing to the image that was picked.
3333
///

packages/image_picker/image_picker/pubspec.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: image_picker
22
description: Flutter plugin for selecting images from the Android and iOS image
33
library, and taking new pictures with the camera.
44
homepage: https://github.com/flutter/plugins/tree/master/packages/image_picker/image_picker
5-
version: 0.6.7+18
5+
version: 0.6.7+19
66

77
flutter:
88
plugin:
@@ -25,7 +25,9 @@ dev_dependencies:
2525
sdk: flutter
2626
integration_test:
2727
path: ../../integration_test
28+
mockito: ^4.1.3
2829
pedantic: ^1.8.0
30+
plugin_platform_interface: ^1.0.3
2931

3032
environment:
3133
sdk: ">=2.1.0 <3.0.0"

packages/image_picker/image_picker/test/image_picker_test.dart

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
import 'package:flutter/services.dart';
66
import 'package:flutter_test/flutter_test.dart';
77
import 'package:image_picker/image_picker.dart';
8+
import 'package:image_picker_platform_interface/image_picker_platform_interface.dart';
9+
import 'package:mockito/mockito.dart';
10+
import 'package:plugin_platform_interface/plugin_platform_interface.dart';
811

912
void main() {
1013
TestWidgetsFlutterBinding.ensureInitialized();
@@ -26,6 +29,15 @@ void main() {
2629
log.clear();
2730
});
2831

32+
test('ImagePicker platform instance overrides the actual platform used',
33+
() {
34+
final ImagePickerPlatform savedPlatform = ImagePickerPlatform.instance;
35+
final MockPlatform mockPlatform = MockPlatform();
36+
ImagePickerPlatform.instance = mockPlatform;
37+
expect(ImagePicker.platform, mockPlatform);
38+
ImagePickerPlatform.instance = savedPlatform;
39+
});
40+
2941
group('#pickImage', () {
3042
test('passes the image source argument correctly', () async {
3143
await picker.getImage(source: ImageSource.camera);
@@ -336,3 +348,7 @@ void main() {
336348
});
337349
});
338350
}
351+
352+
class MockPlatform extends Mock
353+
with MockPlatformInterfaceMixin
354+
implements ImagePickerPlatform {}

packages/url_launcher/url_launcher/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 6.0.0-nullsafety.3
2+
3+
* forceSafariVC should be nullable.
4+
15
## 6.0.0-nullsafety.2
26

37
* Fix outdated links across a number of markdown files ([#3276](https://github.com/flutter/plugins/pull/3276))

packages/url_launcher/url_launcher/lib/url_launcher.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ import 'package:url_launcher_platform_interface/url_launcher_platform_interface.
6262
/// is set to true and the universal link failed to launch.
6363
Future<bool> launch(
6464
String urlString, {
65-
bool forceSafariVC = true,
65+
bool? forceSafariVC,
6666
bool forceWebView = false,
6767
bool enableJavaScript = false,
6868
bool enableDomStorage = false,
@@ -95,7 +95,7 @@ Future<bool> launch(
9595

9696
final bool result = await UrlLauncherPlatform.instance.launch(
9797
urlString,
98-
useSafariVC: forceSafariVC,
98+
useSafariVC: forceSafariVC ?? isWebURL,
9999
useWebView: forceWebView,
100100
enableJavaScript: enableJavaScript,
101101
enableDomStorage: enableDomStorage,

packages/url_launcher/url_launcher/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: url_launcher
22
description: Flutter plugin for launching a URL on Android and iOS. Supports
33
web, phone, SMS, and email schemes.
44
homepage: https://github.com/flutter/plugins/tree/master/packages/url_launcher/url_launcher
5-
version: 6.0.0-nullsafety.2
5+
version: 6.0.0-nullsafety.3
66

77
flutter:
88
plugin:

packages/video_player/video_player/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 2.0.0-nullsafety.6
2+
3+
* Fix `VideoPlayerValue toString()` test.
4+
15
## 2.0.0-nullsafety.5
26

37
* Fix outdated links across a number of markdown files ([#3276](https://github.com/flutter/plugins/pull/3276))

packages/video_player/video_player/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ description: Flutter plugin for displaying inline video with other Flutter
44
# 0.10.y+z is compatible with 1.0.0, if you land a breaking change bump
55
# the version to 2.0.0.
66
# See more details: https://github.com/flutter/flutter/wiki/Package-migration-to-1.0.0
7-
version: 2.0.0-nullsafety.5
7+
version: 2.0.0-nullsafety.6
88
homepage: https://github.com/flutter/plugins/tree/master/packages/video_player/video_player
99

1010
flutter:

packages/video_player/video_player/test/video_player_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,7 @@ void main() {
567567
'VideoPlayerValue(duration: 0:00:05.000000, '
568568
'size: Size(400.0, 300.0), '
569569
'position: 0:00:01.000000, '
570-
'caption: Caption(number: null, start: null, end: null, text: foo), '
570+
'caption: Caption(number: 0, start: 0:00:00.000000, end: 0:00:00.000000, text: foo), '
571571
'buffered: [DurationRange(start: 0:00:00.000000, end: 0:00:04.000000)], '
572572
'isInitialized: true, '
573573
'isPlaying: true, '

0 commit comments

Comments
 (0)