summaryrefslogtreecommitdiff
path: root/core/java/android/content/LoggingContentInterface.java
Commit message (Collapse)AuthorAgeFilesLines
* Extend insert/update/delete to provide extras.Jeff Sharkey2019-11-151-12/+11
| | | | | | | | | | | | | | | | | | | A few releases ago we added ContentResolver.QUERY_ARG_* constants to query() as a new best-practice that will help wean us off raw SQL arguments. (For example, a provider could add their own custom arguments like QUERY_ARG_INCLUDE_PENDING to cause the query to reveal pending items that would otherwise be hidden.) This change expands update() and delete() to accept those arguments. This change also expand insert() to accept extras too, as part of preparing to support an upcoming MediaProvider feature that will let apps place new media "adjacent" to an existing media item. (Sending that adjacent item through extras is cleaner than trying to send it through escaped query parameters.) Bug: 131643582 Test: atest CtsContentTestCases Change-Id: I436296155b9b5f371b4cbe661feaf42070285fcc
* Detailed ContentProvider permissions checks.Jeff Sharkey2019-07-121-0/+13
| | | | | | | | | | | | | | | | | | The new MediaProvider design has an internal dynamic security model based on the value stored in OWNER_PACKAGE_NAME, so the OS always needs to consult the provider when resolving Uri permission grants. Blocking calls from the system process like this are typically discouraged, but this is the best we can do with the limited time left, and there is existing precident with getType(). For now, use "forceUriPermissions" as a proxy for determining when we need to consult the provider directly. Bug: 115619667 Test: atest --test-mapping packages/providers/MediaProvider Test: atest android.appsecurity.cts.ExternalStorageHostTest Change-Id: I1d54feeec93fbb4cf5ff55240ef4eae3a35ed068
* Snapshot arguments before processing.Jeff Sharkey2019-04-021-129/+135
| | | | | | | | | Otherwise the underlying provide may mutate the ContentValues before we log them, meaning we no longer have useful debugging data. Bug: 129487770 Test: none Change-Id: Iba3e90bdf0e51681bbe6b7f1cfd992230aee559c
* Add exception logging to aid debugging.Jeff Sharkey2019-03-171-46/+126
| | | | | | | | Now we log both success and failure of ContentInterface calls. Bug: 128782519 Test: manual Change-Id: I5e81cc6a8ba1460331cad5e085aa01633d6741d6
* Translate selection arguments that are paths.Jeff Sharkey2019-02-281-0/+199
Also migrate MediaProvider logging to more general-purpose location on the ContentProvider.Transport, where we can log exact input/output values to aid debugging. Bug: 124347872 Test: manual Change-Id: I6aba60879ded4e0892d2d1cdd717c23cebaaabd8