diff options
| author | Leon Scroggins <scroggo@google.com> | 2009-06-12 17:07:29 -0400 |
|---|---|---|
| committer | Leon Scroggins <scroggo@google.com> | 2009-06-15 10:16:14 -0400 |
| commit | e372c02c732cf0881c21eb8423454a555e8fc75c (patch) | |
| tree | 72ffa9b0063e5ceba6aa846137c1926de70dd039 /src/com/android/browser/BrowserBookmarksAdapter.java | |
| parent | f26aeabb17d38ed39a0711afcf51694cc69c8d51 (diff) | |
| download | packages_apps_Browser-e372c02c732cf0881c21eb8423454a555e8fc75c.tar.gz packages_apps_Browser-e372c02c732cf0881c21eb8423454a555e8fc75c.tar.bz2 packages_apps_Browser-e372c02c732cf0881c21eb8423454a555e8fc75c.zip | |
Consolidate bookmark add/delete code in the browser.
Driven by a desire to put bookmarking functionality into the title
bar, I have consolidated the code for adding and removing bookmarks.
Diffstat (limited to 'src/com/android/browser/BrowserBookmarksAdapter.java')
| -rw-r--r-- | src/com/android/browser/BrowserBookmarksAdapter.java | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/src/com/android/browser/BrowserBookmarksAdapter.java b/src/com/android/browser/BrowserBookmarksAdapter.java index 09741ae50..c403b44d8 100644 --- a/src/com/android/browser/BrowserBookmarksAdapter.java +++ b/src/com/android/browser/BrowserBookmarksAdapter.java @@ -179,18 +179,7 @@ class BrowserBookmarksAdapter extends BaseAdapter { } mCursor.moveToPosition(position- mExtraOffset); String url = mCursor.getString(Browser.HISTORY_PROJECTION_URL_INDEX); - WebIconDatabase.getInstance().releaseIconForPageUrl(url); - Uri uri = ContentUris.withAppendedId(Browser.BOOKMARKS_URI, mCursor - .getInt(Browser.HISTORY_PROJECTION_ID_INDEX)); - int numVisits = mCursor.getInt(Browser.HISTORY_PROJECTION_VISITS_INDEX); - if (0 == numVisits) { - mContentResolver.delete(uri, null, null); - } else { - // It is no longer a bookmark, but it is still a visited site. - ContentValues values = new ContentValues(); - values.put(Browser.BookmarkColumns.BOOKMARK, 0); - mContentResolver.update(uri, values, null, null); - } + Bookmarks.removeFromBookmarks(null, mContentResolver, url); refreshList(); } |
