summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [automerger skipped] Import translations. DO NOT MERGEBill Yi2019-07-240-0/+0
|\ | | | | | | | | | | | | am: 61f1c5a1ba -s ours am skip reason: subject contains skip directive Change-Id: Ibfc8a581b1c1e12c9335eb485a888fea6f127c18
| * Import translations. DO NOT MERGEBill Yi2019-07-241-2/+2
|/ | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Ifcf4d278763f447d7cb5adb0fc6c4cb33110c3f0
* Import translations. DO NOT MERGEBill Yi2019-07-145-40/+40
| | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Ie766048fd98383e9815d408aad6e9ba4a59841cd
* Set FGS priority for download jobs with visible notifications.Sudheer Shanka2019-07-082-1/+2
|\ | | | | | | | | | | am: bb4439869f Change-Id: I82e9cbed762f2e8917cbef6009d9d7653e88ae9a
| * Set FGS priority for download jobs with visible notifications.Sudheer Shanka2019-07-032-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is what DownloadProvider used to do pre-Q using PRIORITY_FOREGROUND_APP but in Q, we added a new priority to denote FGS but DownloadProvider was still using PRIORITY_FOREGROUND_APP which is now considered a lower priority than FGS and some jobs are getting preempted unnecessarily. So, update DownloadProvider to mark download jobs with FGS priority instead of BFGS priority. This still has some issues but at least this will be kind of reverting DownloadProvider to its pre-Q behavior. Bug: 135982638 Test: manual - Go through DO provisioning a few times and check admin app can be successfully downloaded. Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Change-Id: I020e366845d768da271b860a7ace91318e67d8ae
* | [automerger skipped] Import translations. DO NOT MERGEBill Yi2019-06-250-0/+0
|\| | | | | | | | | | | | | am: f52b0bd3df -s ours am skip reason: subject contains skip directive Change-Id: Ieddf3ab5f63f6572eb23858a98f054e46753bf0e
| * Import translations. DO NOT MERGEBill Yi2019-06-253-5/+5
| | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Ide5c53d86bdaa894b17f6820fa3ebf7dc34db424
* | Import translations. DO NOT MERGEBill Yi2019-06-253-5/+5
| | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Ice58e55ddb5d458c12c7c2d43975cfdb74239aa6
* | Whitelist DownloadManager.ACTION_NOTIFICATION_CLICKED broadcast for ↵Michal Karpinski2019-06-186-1/+34
|\| | | | | | | | | | | | | | | background activity starts am: 45bab8a520 Change-Id: I1ec75e310970b26cf6bf18249e79d2a743003d88
| * Whitelist DownloadManager.ACTION_NOTIFICATION_CLICKED broadcastMichal Karpinski2019-06-186-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | for background activity starts When the DownloadManager download notification is clicked, that broadcast is sent to an app, and the app should be able to open activities from bg for its duration. Bug: 135515407 Test: com.android.providers.downloads.PublicApiFunctionalTest#testNotificationClickedBroadcast Test: manual Change-Id: Ic1691f5bda890ceb2938ac76f4ca2a4c7e7a1160
* | Merge "Delegate deletion of mediaprovider downloads to FileSystemProvider." ↵Sudheer Shanka2019-06-141-10/+2
|\| | | | | | | | | | | | | | | into qt-dev am: c2dc83fb4c Change-Id: I811574ad96e5525ac76918efdff3293881f0404b
| * Merge "Delegate deletion of mediaprovider downloads to FileSystemProvider." ↵Sudheer Shanka2019-06-141-10/+2
| |\ | | | | | | | | | into qt-dev
| | * Delegate deletion of mediaprovider downloads to FileSystemProvider.Sudheer Shanka2019-06-131-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using ContentResolver.delete() directly on MediaProvider doesn't work for uris corresponding to directories. For now, delegate this deletion to FileSystemProvider which will try deleting all entries regex-matching the file path. Bug: 135031305 Test: manual Change-Id: Iac18d0fd0be05758aaf2c03971d716592d96699b
* | | Merge "Import translations. DO NOT MERGE" into qt-r1-devTreeHugger Robot2019-06-149-10/+10
|\ \ \
| * | | Import translations. DO NOT MERGEBill Yi2019-06-139-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: I016980d27b2f13f5a05ce5a7798b3911af0a342a
* | | | [automerger skipped] Import translations. DO NOT MERGE am: 00cced240d -s oursBill Yi2019-06-140-0/+0
|\ \ \ \ | |/ / / |/| / / | |/ / | | | | | | | | | am: 1a7a0ecbc4 -s ours am skip reason: subject contains skip directive Change-Id: Iacb8a7ec806f53fdf2179313af7730f588bffa04
| * | [automerger skipped] Import translations. DO NOT MERGEBill Yi2019-06-140-0/+0
| |\ \ | | |/ | |/| | | | | | | | | | | | | am: 00cced240d -s ours am skip reason: subject contains skip directive Change-Id: I43d16ab9ca453633ff0aefd1b226c7b62a067052
| | * Import translations. DO NOT MERGEBill Yi2019-06-1320-41/+41
| | | | | | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: I7e105e0b1484a04abdba3f72bcc140ff6f3480e3
* | | [automerger skipped] Import translations. DO NOT MERGEBill Yi2019-06-090-0/+0
|\| | | | | | | | | | | | | | | | | | | | am: f66042e2d0 -s ours am skip reason: subject contains skip directive Change-Id: Id37fe9bd3096a9c497126a6f7dbe869b1fe560d4
| * | Import translations. DO NOT MERGEBill Yi2019-06-082-2/+2
| | | | | | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Iee77bf7755e593379fa2b004d916fcb71aa9b3c0
* | | Merge "Delete entries of uninstalled pkgs which haven't started ↵Sudheer Shanka2019-05-291-1/+4
|\| | | | | | | | | | | | | | | | | | | | | | | downloading." into qt-dev am: 1ab3616ee2 Change-Id: I285dde017e503bf2348256cde44e9eeb82f8d1be
| * | Merge "Delete entries of uninstalled pkgs which haven't started ↵TreeHugger Robot2019-05-291-1/+4
| |\ \ | | | | | | | | | | | | downloading." into qt-dev
| | * | Delete entries of uninstalled pkgs which haven't started downloading.Sudheer Shanka2019-05-291-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: 133741222 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Change-Id: I00796e5ff8e9b30d153490f997fb7c776b600e57
* | | | [automerger skipped] Import translations. DO NOT MERGEBill Yi2019-05-240-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | am: 36432867c1 -s ours am skip reason: subject contains skip directive Change-Id: I8daa5eb5ace287c27b2bd27ae069686e23439ba7
| * | | Import translations. DO NOT MERGEBill Yi2019-05-247-8/+8
| |/ / | | | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: If8e7af64199b4c977195c4ee66919aa9045580b0
* | | Merge "Don't use linked mediastore uris in DownloadProvider operations." ↵Sudheer Shanka2019-05-212-188/+78
|\| | | | | | | | | | | | | | | | | | | | | | | into qt-dev am: 7221fab459 Change-Id: If2798954f856ad280c147d252fd2f3d1c955de17
| * | Merge "Don't use linked mediastore uris in DownloadProvider operations." ↵Sudheer Shanka2019-05-222-188/+78
| |\ \ | | | | | | | | | | | | into qt-dev
| | * | Don't use linked mediastore uris in DownloadProvider operations.Sudheer Shanka2019-05-212-188/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When MediaProvider db gets recreated, all the media content ids get renumbered. It's possible that when DownloadProvider is trying to delete an entry, it is holding onto a invalid mediastore uri. So, don't use linked mediastore uris in DownloadProvider operations. Also, revoke any prior uri grants of media content from DownloadStorageProvider. Bug: 132087334 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Test: atest cts/hostsidetests/appsecurity/src/android/appsecurity/cts/AppSecurityTests.java Change-Id: If6fb479da7e937ecdfa23136811f3456f7bcd75c
* | | | Update DownloadStorageProvider for trailing '/' in RELATIVE_PATHshafik2019-05-211-2/+2
|\| | | | |/ / |/| | | | | | | | | | | am: 333602c3ab Change-Id: I3a573cfd4cd1dc8dfed3ef949eee965527a337f3
| * | Update DownloadStorageProvider for trailing '/' in RELATIVE_PATHshafik2019-05-211-2/+2
|/ / | | | | | | | | | | | | | | Add trailing '/' to RELATIVE_PATH selection args in db queries. Test: atest --test-mapping packages/providers/MediaProvider Bug:132631113 Change-Id: Ib8887dbb0e49dc76df6e4292d36e8f5948f256c5
* | Allow files to be created under download dirs from MediaStore.Sudheer Shanka2019-05-131-3/+8
| | | | | | | | | | | | | | | | | | | | Also, allow files to be renamed if they are not in pending state and trigger a media scan when a new file is created. Bug: 127844893 Bug: 127802924 Test: manual Change-Id: Iec14c5d349930c5fae54ac3346ee5cf11fcda387
* | Update behavior of setDestinationInExternalPublicDir().Sudheer Shanka2019-05-094-106/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow apps to use DownloadManager for downloading files into known public dirs other than "Download" but don't include them as part of Downloads collection. Bug: 132136431 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Change-Id: I62d4f810c71c9231e2b6d8e276a0a1e326382e14
* | Merge "Don't log when checking if a filepath belongs to certain dirs." into ↵Sudheer Shanka2019-05-072-4/+6
|\ \ | | | | | | | | | qt-dev
| * | Don't log when checking if a filepath belongs to certain dirs.Sudheer Shanka2019-05-062-4/+6
| | | | | | | | | | | | | | | | | | Bug: 131693243 Test: n/a Change-Id: Iae700a00f1bca78f823a3f8b732d86d380942828
* | | Merge "Fix DownloadProviderTests." into qt-devSudheer Shanka2019-05-062-3/+11
|\ \ \
| * | | Fix DownloadProviderTests.Sudheer Shanka2019-05-012-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 131693243 Test: atest DownloadProviderTests Change-Id: Iefdd7690e45351c2dbdac1cff49f0d11aeba0fb0
* | | | Import translations. DO NOT MERGEBill Yi2019-05-034-14/+14
| |/ / |/| | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: I6c763081dde011763dae9f978a9f8a61abc4f0cf
* | | Merge "Update API impl of ↵TreeHugger Robot2019-05-013-8/+66
|\| | | | | | | | | | | allowScanningByMediaScanner/setVisibleInDownloadsUi." into qt-dev
| * | Update API impl of allowScanningByMediaScanner/setVisibleInDownloadsUi.Sudheer Shanka2019-04-303-8/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These APIs javadocs have been updated earlier to indicate how they behave on Q, now updating impl of those APIs to match with javadocs. Bug: 131693243 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Change-Id: I8fb43806b27fefaffc510fde5424b48e3ff56d20
* | | Merge "Skip restrictions if DownloadProvider itself is trying to add a ↵Sudheer Shanka2019-04-301-1/+3
|\ \ \ | |/ / |/| | | | | download." into qt-dev
| * | Skip restrictions if DownloadProvider itself is trying to add a download.Sudheer Shanka2019-04-301-1/+3
| | | | | | | | | | | | | | | | | | | | | Fixes: 131624566 Fixes: 131610977 Test: manual Change-Id: I771ff8a39ac977c510f1785c3cb61e859bbb0c0f
* | | Make downloads deletion on owner uninstall consistent.Sudheer Shanka2019-04-283-24/+98
|/ / | | | | | | | | | | | | | | | | | | | | | | When an app is uninstalled, it's downloads in top-level download directory will be orphaned and the rest will be deleted. Bug: 126064843 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Change-Id: Iad430f7ec14f306ed859a3b30db9eea29a23c19d
* | Merge "Import translations. DO NOT MERGE" into qt-devTreeHugger Robot2019-04-253-4/+4
|\ \
| * | Import translations. DO NOT MERGEBill Yi2019-04-243-4/+4
| | | | | | | | | | | | | | | | | | | | | Auto-generated-cl: translation import Bug: 64712476 Change-Id: Ie89705d17abb8825621dd7213ced155259f81367
* | | Update behavior of some DownloadManager APIs.Sudheer Shanka2019-04-232-130/+194
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | -- Update behavior of DownloadManager.setDestinationUri(), DownloadManager.setDestinationInExternalPublicDir() and DownloadManager.addCompletedDownload() based on the latest storage re-design. Essentially, going forward these APIs will only allow downloading files into package owned dirs or the top-level Download dir. -- Allow some system components to specify MediaColumns.OWNER_PACKAGE_NAME when inserting items into MediaProvider. -- Don't copy DownloadManager.COLUMN_TITLE to MediaProvider. DownloadProvider and MediaProvider have different constraints around "title" and there isn't really a need to keep these in sync. -- Sanity check file download paths hinted by apps. -- Remove sandbox related logic in DownloadProvider. Bug: 120879208 Bug: 128630262 Bug: 130797842 Test: manual Test: atest DownloadProviderTests Test: atest cts/tests/app/src/android/app/cts/DownloadManagerTest.java Test: atest cts/tests/app/DownloadManagerLegacyTest/src/android/app/cts/DownloadManagerLegacyTest.java Test: atest cts/tests/app/DownloadManagerApi28Test/src/android/app/cts/DownloadManagerApi28Test.java Test: atest cts/tests/tests/provider/src/android/provider/cts/MediaStore* Change-Id: I8f4569f91f2c0567ff892a2107183cd11b7065a7
* | Remove logic of including downloads dirs in pkg sandboxes.Sudheer Shanka2019-04-181-49/+29
| | | | | | | | | | | | Bug: 130427780 Test: manual Change-Id: I3a3c98dfa769ea6e2dac1fb3d92148489c04f3a1
* | Merge "Update DownloadStorageProvider to use MediaStore.DownloadColumns." ↵Sudheer Shanka2019-04-182-55/+112
|\ \ | | | | | | | | | into qt-dev
| * | Update DownloadStorageProvider to use MediaStore.DownloadColumns.Sudheer Shanka2019-04-172-55/+112
| | | | | | | | | | | | | | | | | | | | | Bug: 130232195 Test: manual Test: atest --test-mapping packages/providers/MediaProvider Change-Id: I77cd8b38c729aaaa2df138c3bc30049dba68b693
* | | Merge "Fix issue of downloaded apks not getting installed." into qt-devSudheer Shanka2019-04-181-0/+1
|\ \ \
| * | | Fix issue of downloaded apks not getting installed.Sudheer Shanka2019-04-171-0/+1
| |/ / | | | | | | | | | | | | | | | Fixes: 130759236 Test: manual Change-Id: I8791cd3261f6a0f85cb5bc0e6d13a656fa7b1e8b