Skip to content

Commit

Permalink
Fix bug: should use sdkInt as the property to compare (#3865)
Browse files Browse the repository at this point in the history
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

Task/Issue URL: https://app.asana.com/0/0/1205952041108121/f 

### Description
Bug fix: 
- replaces `appBuildConfig.versionCode` with `appBuildConfig.sdkInt` in
the OS version guard.

### Steps to test this PR

QA optional
  • Loading branch information
CDRussell authored Nov 14, 2023
1 parent 1a0dfe4 commit 51f88d0
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ class RealSystemAutofillServiceSuppressor @Inject constructor(

@SuppressLint("NewApi")
override fun suppressAutofill(webView: WebView?) {
if (appBuildConfig.versionCode >= Build.VERSION_CODES.O) {
if (appBuildConfig.sdkInt >= Build.VERSION_CODES.O) {
webView?.context?.getSystemService(AutofillManager::class.java)?.cancel()
}
}
Expand Down

0 comments on commit 51f88d0

Please sign in to comment.