Request: provide more cases that apps could reach the clipboard, and not just system-apps, keyboard-apps and focused-apps [151294241]

Obsolete

Bug

[AOSP] assigned

adexe s nau

Status Update

No update yet.

Description

lb...@gmail.com

created issue #1

Mar 13, 2020 02:47AM

The exceptions of the clipbaord monitoring is very narrow, not allowing any clipboard app to adjust to the changes:

https://developer.android.com/about/versions/10/privacy/changes#clipboard-data

"Unless your app is the default input method editor (IME) or is the app that currently has focus, your app cannot access clipboard data on Android 10 or higher.
"

Please think of more cases that apps could still handle.

Maybe accessibility apps (those can already get a lot on the screen)?
Assistant apps (those can pre-fetch data about the copied text to search it sooner)?
Apps with SAW permission or any that can start on its own (those can already appear on their own and get the clipboard data) :

https://developer.android.com/guide/components/activities/background-starts#exceptions

Comments

am...@google.com <am...@google.com> Mar 13, 2020 04:50AM

Assigned to am...@google.com.

am...@google.com <am...@google.com> #2Mar 13, 2020 06:46AM

Thank you for the report, and sorry for any inconvenience. While reproducing the issue, one workaround I found is: -keeppackagenames com.example.customview.extra** (i.e., double asterisks, instead of single).

https://www.guardsquare.com/en/products/proguard/manual/usage#filters

R8 currently regards <package_filter> in that directive as <class_filter>. That's why single asterisk---a class name not containing the package separator---doesn't work for now, whereas double asterisks---matches a class name, possibly containing any number of package separators---works. The caveat is, use of, e.g., extra**, will keep all the sub packages as well. :( We'll prepare the fix soon.

ad...@google.com <ad...@google.com> Aug 19, 2021 03:25PM

Status: Won't Fix (Obsolete)

lb...@gmail.com <lb...@gmail.com> #3Aug 20, 2021 07:32PM

Project: r8
Branch: master

commit a9a9e7d1231726094c6e16c2cef1bad887da3bba
Author: Jinseong Jeon <jsjeon@google.com>
Date: Tue Apr 09 11:49:28 2019

Reproduce

b/130135768

: keeppackagenames with asterisks

Bug: 130135768
Change-Id: I5ab1cdb5148a570a63bdd488358759bb33da700a

M src/test/java/com/android/tools/r8/TestShrinkerBuilder.java
A src/test/java/com/android/tools/r8/naming/KeepPackageNamesTest.java
A src/test/java/com/android/tools/r8/naming/keeppackagenames/Top.java
A src/test/java/com/android/tools/r8/naming/keeppackagenames/sub/SubClass.java

https://r8-review.googlesource.com/36860

bu...@gmail.com <bu...@gmail.com> #4Sep 17, 2021 01:45AM

Project: r8
Branch: master

commit 391e2992aee375adf46edbb5e7b098556c8604ed
Author: Søren Gjesse <sgjesse@google.com>
Date: Fri Apr 12 11:05:43 2019

New implementation of -keeppackagenames

This implementation collects the list of package names given for
-keeppackagenames and matches against that in the minifier.

The previous implementation created class matching rules to piggybag
on the existing matching.

Bug: 130135768
Change-Id: I5a341e145747a5dec8788a066a0c67d4e259ec77

M src/main/java/com/android/tools/r8/graph/DexType.java
M src/main/java/com/android/tools/r8/naming/ClassNameMinifier.java
M src/main/java/com/android/tools/r8/shaking/ProguardConfiguration.java
M src/main/java/com/android/tools/r8/shaking/ProguardConfigurationParser.java
D src/main/java/com/android/tools/r8/shaking/ProguardKeepPackageNamesRule.java
A src/main/java/com/android/tools/r8/shaking/ProguardPackageMatcher.java
A src/main/java/com/android/tools/r8/shaking/ProguardPackageNameList.java
M src/main/java/com/android/tools/r8/shaking/RootSetBuilder.java
M src/test/java/com/android/tools/r8/naming/KeepPackageNamesTest.java
M src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java
A src/test/java/com/android/tools/r8/shaking/ProguardPackageNameMatcherTest.java

https://r8-review.googlesource.com/36944

Issue 151294241