Skip to content

[pull] androidx-main from androidx:androidx-main#483

Merged
pull[bot] merged 9 commits intowarting:androidx-mainfrom
androidx:androidx-main
Sep 15, 2025
Merged

[pull] androidx-main from androidx:androidx-main#483
pull[bot] merged 9 commits intowarting:androidx-mainfrom
androidx:androidx-main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Sep 15, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

WenHung_Teng and others added 9 commits September 12, 2025 18:48
The `setCameraOpenRetryMaxTimeoutInMillis` value from `CameraXConfig`
was not being applied correctly for the CameraPipe backend.

An inverted boolean check in `CameraFactoryProvider` caused the timeout
to be ignored unless it was set to the default value of -1.

This change corrects the conditional check to properly handle the `-1L` sentinel value.

Bug: 444627589
Test: CameraStateRobolectricTest covers this behavior.
Change-Id: Ice310e397c5597131a0a5aa8cc3f6de6389b8f0d
Add the new `TransitionState` sealed class. This API will
represent the physical state of a navigation gesture, either
`Idle` or `InProgress`.

This class intentionally contains no generics and is separate from
navigation history (`Info`). It is the foundational step in
refactoring the dispatcher to expose two distinct global state
flows (`TransitionState` and `HistoryState`) instead of the old
merged generic state.

RelNote: "Introduce the new `TransitionState` sealed class. This will
serve as the core API for observing gesture state, separate from
navigation history."

Bug: 444734264
Test: existing passes
Change-Id: Id4beb114ebc4ca33b8810de2d1cf9d57f7ec2336
Add a new public, read-only `transitionState` property to the
`NavigationEventHandler`.

The handler is now responsible for updating its own `transitionState`
whenever its gesture callbacks (`doOn...Started`, `...Progressed`,
`...Completed`, etc.) are invoked. The state flips to `InProgress`
when a gesture starts and returns to `Idle` when it terminates.

This makes the active handler the source of truth for the current
gesture state. The processor will read this public property to
drive the new global `dispatcher.transitionState` flow.

RelNote: "Add a new public, read-only `transitionState: TransitionState`
property to `NavigationEventHandler`. Handlers now maintain their own
transition state, which external systems can observe."

Bug: 444734264
Test: existing passes
Change-Id: I9acd2b09de191f33cb200169fc1b4d3bb74dd17a
The Camera2ImplConfig in CameraPipe integration leaves options in the
global namespace (of the package), which is different than Camera2 which
places these constants in Camera2ImplConfig.

Make them API compatible by putting these constants inside the companion
object marked with @JvmField

Bug: 435076868
Test: Unit tests
Change-Id: If1fb225d613c3a2bd1132c7184f29a565e307006
Fix incorrect KDoc in Camera2CameraInfo. A type parameter doesn't need
to be enclosed with '<>', and this is causing exceptions in the static
analyzer.

Bug: 435076868
Test: :docs-tip-of-tree:docs
Change-Id: I51f8c1f035f5faed50fa511bafddb17200baf4b3
* changes:
  Fix incorrect KDoc in Camera2CameraInfo
  Make Camera2ImplConfig API compatible
@pull pull bot locked and limited conversation to collaborators Sep 15, 2025
@pull pull bot added the ⤵️ pull label Sep 15, 2025
@pull pull bot merged commit 6cc9eea into warting:androidx-main Sep 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants