summaryrefslogtreecommitdiffstats
path: root/src/com/android/browser/provider
diff options
context:
space:
mode:
authorJohn Reck <jreck@google.com>2011-03-03 15:30:26 -0800
committerJohn Reck <jreck@google.com>2011-03-03 15:53:19 -0800
commit57e39f9e46091750c8a07895531446a7f0efdb21 (patch)
tree1c3ab3a9af062201482db8dd7e727c50670be40d /src/com/android/browser/provider
parentb52c0d8be3486e0e2d89a39c521daa5020605ac2 (diff)
downloadpackages_apps_Browser-57e39f9e46091750c8a07895531446a7f0efdb21.zip
packages_apps_Browser-57e39f9e46091750c8a07895531446a7f0efdb21.tar.gz
packages_apps_Browser-57e39f9e46091750c8a07895531446a7f0efdb21.tar.bz2
Fix isValidParent check
Bug: 3506038 Lookup the parent itself, not the children of the parent in isValidParent check. Also, adds testing to watch for regressions Change-Id: I59f00094f862f2e9cd4b4f9fd92d370b87e612e5
Diffstat (limited to 'src/com/android/browser/provider')
-rw-r--r--src/com/android/browser/provider/BrowserProvider2.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/com/android/browser/provider/BrowserProvider2.java b/src/com/android/browser/provider/BrowserProvider2.java
index 94b1651..c6b1bc7 100644
--- a/src/com/android/browser/provider/BrowserProvider2.java
+++ b/src/com/android/browser/provider/BrowserProvider2.java
@@ -1236,7 +1236,7 @@ public class BrowserProvider2 extends SQLiteContentProvider {
private boolean isValidParent(String accountType, String accountName,
long parentId) {
- Uri uri = Bookmarks.buildFolderUri(parentId);
+ Uri uri = ContentUris.withAppendedId(Bookmarks.CONTENT_URI, parentId);
Cursor c = query(uri,
new String[] { Bookmarks.ACCOUNT_NAME, Bookmarks.ACCOUNT_TYPE },
null, null, null);