Integer.MAX_VALUE does not work for ActivityResultContracts.PickMultipleVisualMedia [249182130]

Fixed

Bug

Status Update

No update yet.

Description

vi...@nobel-systems.com

created issue #1

Sep 27, 2022 03:26AM

Component used: Activity

Version used: 1.6.0

Devices/Android versions reproduced on: Samsung Galaxy S20+ Android 12 (with August 2022 Google Play system update, photo picker is confirmed working)

For ActivityResultContracts.PickMultipleVisualMedia, you use Integer.MAX_VALUE as an overload for the maximum number of items the photo picker is able to select for versions Android 12 and below (as MediaStore.getPickImagesMaxLimit() is only available for TIRAMISU and above).

However, using this value causes the photo picker to not be able to start/be launched.

I tried changing the maxItems value to another value (I tried 100) and the photo picker started to work.

Comments

il...@google.com <il...@google.com> Sep 27, 2022 10:41PM

Assigned to yr...@google.com.

yr...@google.com <yr...@google.com> #2Sep 30, 2022 03:51PM

Accepted by yr...@google.com.

Project: platform/frameworks/support
Branch: androidx-main

commit c46d3e79b2ea8b839358f0215b0a432ef8fad7a2
Author: Clara Fok <clarafok@google.com>
Date: Fri Nov 04 15:23:25 2022

Convert androidx.activity.Cancellable to kotlin

Test: ./gradlew activity:activity:cC
Bug: 257291701
Change-Id: I057f3e54811e7399e40083ef5e3915adc4fa425d

M activity/activity/src/main/java/androidx/activity/Cancellable.kt

https://android-review.googlesource.com/2288554

ap...@google.com <ap...@google.com> #3Oct 12, 2022 02:57AM

Project: platform/frameworks/support
Branch: androidx-main

commit 7fd8a3206a49baefee8340f0839fcd4de7716788
Author: Clara Fok <clarafok@google.com>
Date: Fri Nov 04 15:13:23 2022

Rename androidx.activity.Cancellable from .java to .kt

Test: n/a
Bug:257291701
Change-Id: I7f2ff8ce79af41c6437fb9a0f1be90e8990495e8

M activity/activity/src/main/java/androidx/activity/Cancellable.kt

https://android-review.googlesource.com/2288553

il...@google.com <il...@google.com> #4Oct 12, 2022 07:59PM

Marked as fixed, reassigned to jb...@google.com.

Project: platform/frameworks/support
Branch: androidx-main

commit 4d21e0a45f19ed238ef30756f576930af07449c2
Author: Clara Fok <clarafok@google.com>
Date: Fri Nov 04 13:25:08 2022

Convert ActivityResultCallback to kotlin

Including clean up in test usages of this interface

Test: ./gradlew activity:activity:cC
Bug: 257291701
Relnote: Converted ActvitiyResultCallback to kotlin. Maintains source and binary comptability.
Change-Id: Ifc5e5e1a9a7ed4e281007830fb8d4c74237c63ce

M activity/activity/api/current.txt
M activity/activity/api/public_plus_experimental_current.txt
M activity/activity/api/restricted_current.txt
M activity/activity/src/androidTest/java/androidx/activity/result/ActivityResultRegistryTest.kt
M activity/activity/src/main/java/androidx/activity/result/ActivityResultCallback.kt

https://android-review.googlesource.com/2288334

na...@google.com <na...@google.com> #5Oct 24, 2022 01:37PM

Project: platform/frameworks/support
Branch: androidx-main

commit ae5bcbc9174b9767f143bfd9629a69fcd3ed325d
Author: Clara Fok <clarafok@google.com>
Date: Fri Nov 04 13:06:36 2022

Rename ActivityResultCallback from .java to .kt

Test: n/a
Bug: 257291701
Change-Id: I43a8c76cb51e3eee4a1ec2cb6f0128753d85965d

M activity/activity/src/main/java/androidx/activity/result/ActivityResultCallback.kt

https://android-review.googlesource.com/2288333

al...@gmail.com <al...@gmail.com> #6Nov 3, 2022 12:21AM

Comment has been deleted.

Message last modified on Nov 3, 2022 04:37PM

fv...@gmail.com <fv...@gmail.com> #7Mar 5, 2023 06:26AM

Comment has been deleted.

Message last modified on Mar 6, 2023 07:22PM

fv...@gmail.com <fv...@gmail.com> #8Mar 5, 2023 06:28AM

Comment has been deleted.

Message last modified on Mar 6, 2023 07:22PM

fv...@gmail.com <fv...@gmail.com> #9Mar 5, 2023 06:29AM

Comment has been deleted.

Message last modified on Mar 6, 2023 07:22PM

ar...@gmail.com <ar...@gmail.com> #10Jun 25, 2023 05:44PM

Project: platform/frameworks/support
Branch: androidx-main

commit 453e6f7acd7b7a1310487fc912f78c0fd2e72dd3
Author: Ian Lake <ilake@google.com>
Date: Tue Jan 24 23:43:12 2023

Convert OnBackPressedDispatcher to Kotlin

Relnote: "`OnBackPressedDispatcher` is now written in Kotlin."
Test: ./gradlew checkApi
BUG: 257291701
Change-Id: I18ac79d3cd20fc29159542b729935d3b8d2e4506

M activity/activity/api/current.txt
M activity/activity/api/public_plus_experimental_current.txt
M activity/activity/api/restricted_current.txt
M activity/activity/src/main/java/androidx/activity/OnBackPressedDispatcher.kt

https://android-review.googlesource.com/2401941

an...@gmail.com <an...@gmail.com> #11Apr 1, 2024 02:26AM

Project: platform/frameworks/support
Branch: androidx-main

commit 57dd231f03ef3f5d107f7c9be9b0f5dd200d7ec4
Author: Ian Lake <ilake@google.com>
Date: Tue Jan 24 23:01:48 2023

Rename OnBackPressedDispatcher.java to .kt

Test: n/a
BUG: 257291701
Change-Id: Id490de8b28b7d2a680c76c822f564fdadd5eb5f1

M activity/activity/src/main/java/androidx/activity/OnBackPressedDispatcher.kt

https://android-review.googlesource.com/2401940

al...@gmail.com <al...@gmail.com> #12May 25, 2024 10:10AM

Project: platform/frameworks/support
Branch: androidx-main

commit 76fcc8653339d275c9bfad10e4c68cd9c8ef0dfd
Author: Ian Lake <ilake@google.com>
Date: Fri Jan 27 00:45:07 2023

Update wear module with OnBackPressedDispatcherOwner

With the move of OnBackPressedDispatcherOwner to Kotlin,
implementations must now override a property rather
than implement a getter.

Test: ./gradlew bOS
BUG: 257291701
Change-Id: Iaf4f466f38a23c5522de019d38080b04c9f7b7ff

M wear/compose/compose-navigation/build.gradle
M wear/compose/compose-navigation/src/androidTest/kotlin/androidx/wear/compose/navigation/SwipeDismissableNavHostTest.kt

https://android-review.googlesource.com/2405513

al...@gmail.com <al...@gmail.com> #13May 25, 2024 10:11AM

Project: platform/frameworks/support
Branch: androidx-main

commit a63f1351aab5e387a829711f14de221274119bcd
Author: Ian Lake <ilake@google.com>
Date: Thu Jan 26 22:51:46 2023

Update navigation module with OnBackPressedDispatcherOwner

With the move of OnBackPressedDispatcherOwner to Kotlin,
implementations must now override a property rather
than implement a getter.

Test: ./gradlew bOS
BUG: 257291701
Change-Id: Iad795301ff2e6a16b42be6b85e6370ac399e43fd

M navigation/navigation-compose/build.gradle
M navigation/navigation-compose/src/androidTest/java/androidx/navigation/compose/NavHostTest.kt

https://android-review.googlesource.com/2404345

al...@gmail.com <al...@gmail.com> #14May 25, 2024 10:11AM

Project: platform/frameworks/support
Branch: androidx-main

commit fa9a4dd728bc575c6e3536f2d5f9564f50b8eeee
Author: Ian Lake <ilake@google.com>
Date: Thu Jan 26 19:33:26 2023

Update compose module with OnBackPressedDispatcherOwner

With the move to OnBackPressedDispatcherOwner to Kotlin,
implementations must now override a property rather
than implement a getter.

Test: ./gradlew bOS
BUG: 257291701
Change-Id: If45fa4c4624a0a71745f4f11f133a3eddcf4dddf

M compose/ui/ui-tooling/build.gradle
M compose/ui/ui-tooling/src/androidMain/kotlin/androidx/compose/ui/tooling/ComposeViewAdapter.kt

https://android-review.googlesource.com/2404338

al...@gmail.com <al...@gmail.com> #15May 25, 2024 10:11AM

Project: platform/frameworks/support
Branch: androidx-main

commit 322bd9508ff202efcdb765778d76baa5fe98795b
Author: Ian Lake <ilake@google.com>
Date: Thu Jan 26 19:05:37 2023

Convert OnBackPressedDispatcherOwner to Kotlin

Relnote: "`OnBackPressedDispatcherOwner` is now written
in Kotlin. This is a **source incompatible change** for those
classes written in Kotlin - you must now override the
`onBackPressedDispatcher` property rather than implementing
the previous `getOnBackPressedDispatcher` function."
Test: ./gradlew checkApi
BUG: 257291701

Change-Id: Ia277dc57731597b869c4e86d3f8c13896a6ce56f

M activity/activity-compose/src/androidTest/java/androidx/activity/compose/BackHandlerTest.kt
M activity/activity-compose/src/androidTest/java/androidx/activity/compose/BackPressedDispatcherOwnerTest.kt
M activity/activity/api/current.txt
M activity/activity/api/public_plus_experimental_current.txt
M activity/activity/api/restricted_current.txt
M activity/activity/src/androidTest/java/androidx/activity/ViewTreeOnBackPressedDispatcherTest.kt
M activity/activity/src/main/java/androidx/activity/ComponentDialog.kt
M activity/activity/src/main/java/androidx/activity/OnBackPressedCallback.kt
M activity/activity/src/main/java/androidx/activity/OnBackPressedDispatcherOwner.kt

https://android-review.googlesource.com/2404337

al...@gmail.com <al...@gmail.com> #16May 25, 2024 10:11AM

Project: platform/frameworks/support
Branch: androidx-main

commit c0ae191b22d3b679c0282ebfad7cc35670afc3b4
Author: Ian Lake <ilake@google.com>
Date: Thu Jan 26 18:24:49 2023

Rename OnBackPressedDispatcherOwner.java to .kt

Part of the Kotlin conversion process.

Test: n/a
BUG: 257291701
Change-Id: Ic0b969a9f98d461ec48195309989f740891ebb9f

M activity/activity/src/main/java/androidx/activity/OnBackPressedDispatcherOwner.kt

https://android-review.googlesource.com/2404336

al...@gmail.com <al...@gmail.com> #17May 25, 2024 10:11AM

Project: platform/frameworks/support
Branch: androidx-main

commit c83511c81d5fd0c87577180301ca1b1231d24c13
Author: Ian Lake <ilake@google.com>
Date: Mon Jan 30 18:44:32 2023

Update compose module with ActivityResultRegistryOwner

With the move of ActivityResultRegistryOwner to Kotlin,
implementations must now override a property rather
than implement a getter.

Test: ./gradlew bOS
BUG: 257291701
Change-Id: I490b9b69d5bd4d9c608ca8d3c9a196d3e70fe801

M compose/ui/ui-tooling/src/androidMain/kotlin/androidx/compose/ui/tooling/ComposeViewAdapter.kt

https://android-review.googlesource.com/2407774

al...@gmail.com <al...@gmail.com> #18May 25, 2024 10:12AM

Project: platform/frameworks/support
Branch: androidx-main

commit 74e5a4b559f319808f1addeb4d1bc46d4267ccdb
Author: Ian Lake <ilake@google.com>
Date: Mon Jan 30 18:38:17 2023

Convert ActivityResultRegistryOwner to Kotlin

Relnote: "`ActivityResultRegistryOwner` is now written
in Kotlin. This is a **source incompatible change**
for those classes written in Kotlin - you must now
override the `activityResultRegistry` property rather
than implementing the previous
`getActivityResultRegistry()` function."
Test: ./gradlew checkApi
BUG: 257291701

Change-Id: I0b00efc873c16e5b6d26e9ebad889f00a8ddb6b3

M activity/activity-compose/src/androidTest/java/androidx/activity/compose/ActivityResultRegistryTest.kt
M activity/activity/api/current.txt
M activity/activity/api/public_plus_experimental_current.txt
M activity/activity/api/restricted_current.txt
M activity/activity/src/main/java/androidx/activity/result/ActivityResultRegistryOwner.kt

https://android-review.googlesource.com/2407773

al...@gmail.com <al...@gmail.com> #19May 25, 2024 10:12AM

Project: platform/frameworks/support
Branch: androidx-main

commit 7f1a7dcb8a6eb73cc3a54ab161a2c8bfebea48ca
Author: Ian Lake <ilake@google.com>
Date: Mon Jan 30 18:34:24 2023

Rename ActivityResultRegistryOwner.java to .kt

Part of the Kotlin conversion process

Test: n/a
BUG: 257291701
Change-Id: I2e786c6192e2747bf91105f4a1c18703baa0dda7

M activity/activity/src/main/java/androidx/activity/result/ActivityResultRegistryOwner.kt

https://android-review.googlesource.com/2407772

al...@gmail.com <al...@gmail.com> #20May 25, 2024 10:12AM

Project: platform/frameworks/support
Branch: androidx-main

commit c2189c5497529ac8cb8d7e01ac1605c72e029ef2
Author: Ian Lake <ilake@google.com>
Date: Tue Jan 31 00:21:00 2023

Convert IntentSenderRequest to Kotlin

Relnote: "`IntentSenderRequest` is now written in Kotlin."
Test: ./gradlew checkApi
BUG: 257291701
Change-Id: Ib7e74b794dc11ca4ad53bb6145c79f0f63d9c69d

A activity/activity/api/current.ignore
M activity/activity/api/current.txt
M activity/activity/api/public_plus_experimental_current.txt
A activity/activity/api/restricted_current.ignore
M activity/activity/api/restricted_current.txt
M activity/activity/src/main/java/androidx/activity/result/IntentSenderRequest.kt

https://android-review.googlesource.com/2410754

al...@gmail.com <al...@gmail.com> #21May 25, 2024 10:12AM

Project: platform/frameworks/support
Branch: androidx-main

commit 8acdae64bc8e603c57a9d50422592b48a063d6d0
Author: Ian Lake <ilake@google.com>
Date: Mon Jan 30 23:55:36 2023

Rename IntentSenderRequest.java to .kt

Part of the Kotlin conversion process

Test: n/a
BUG: 257291701
Change-Id: I296f10e0a69e68e618a72aa447d08ab7f6daf518

M activity/activity/src/main/java/androidx/activity/result/IntentSenderRequest.kt

https://android-review.googlesource.com/2410752

Issue 249182130

Description

Issue summary

Comments

il...@google.com <il...@google.com> Sep 27, 2022 10:41PM

yr...@google.com <yr...@google.com> #2Sep 30, 2022 03:51PM

ap...@google.com <ap...@google.com> #3Oct 12, 2022 02:57AM

il...@google.com <il...@google.com> #4Oct 12, 2022 07:59PM

na...@google.com <na...@google.com> #5Oct 24, 2022 01:37PM

al...@gmail.com <al...@gmail.com> #6Nov 3, 2022 12:21AM

fv...@gmail.com <fv...@gmail.com> #7Mar 5, 2023 06:26AM

fv...@gmail.com <fv...@gmail.com> #8Mar 5, 2023 06:28AM

fv...@gmail.com <fv...@gmail.com> #9Mar 5, 2023 06:29AM

ar...@gmail.com <ar...@gmail.com> #10Jun 25, 2023 05:44PM

an...@gmail.com <an...@gmail.com> #11Apr 1, 2024 02:26AM

al...@gmail.com <al...@gmail.com> #12May 25, 2024 10:10AM

al...@gmail.com <al...@gmail.com> #13May 25, 2024 10:11AM

al...@gmail.com <al...@gmail.com> #14May 25, 2024 10:11AM

al...@gmail.com <al...@gmail.com> #15May 25, 2024 10:11AM

al...@gmail.com <al...@gmail.com> #16May 25, 2024 10:11AM

al...@gmail.com <al...@gmail.com> #17May 25, 2024 10:11AM

al...@gmail.com <al...@gmail.com> #18May 25, 2024 10:12AM

al...@gmail.com <al...@gmail.com> #19May 25, 2024 10:12AM

al...@gmail.com <al...@gmail.com> #20May 25, 2024 10:12AM

al...@gmail.com <al...@gmail.com> #21May 25, 2024 10:12AM

Add comment

Issue metadata