commit c112990af09d3e15f8503b863f1d8088c258995e
author Leon Scroggins <scroggo@google.com> Wed Dec 08 15:28:33 2010 -0500
committer Leon Scroggins <scroggo@google.com> Wed Dec 08 15:28:33 2010 -0500
tree d0e46d2058f1f8335fbf549742b9c334551aa722
parent a941f6e87e73c15219042714745d1897cbd13582

Do not show allow placing a folder inside itself.

Bug:3263141
Change-Id: I0e070fbab798fe8c439b8ef0e6785b4a06e29466

src/com/android/browser/AddBookmarkPage.java

diff --git a/src/com/android/browser/AddBookmarkPage.java b/src/com/android/browser/AddBookmarkPage.java
index 24a8cd5..2bfcbb4 100644
--- a/src/com/android/browser/AddBookmarkPage.java
+++ b/src/com/android/browser/AddBookmarkPage.java
@@ -339,11 +339,16 @@
                         BrowserContract.Bookmarks.TITLE,
                         BrowserContract.Bookmarks.IS_FOLDER
                 };
+                String where = BrowserContract.Bookmarks.IS_FOLDER + " != 0";
+                if (mEditingFolder) {
+                    where += " AND " + BrowserContract.Bookmarks._ID + " != "
+                            + mMap.getLong(BrowserContract.Bookmarks._ID);
+                }
                 return new CursorLoader(this,
                         BrowserContract.Bookmarks.buildFolderUri(
                         mCurrentFolder),
                         projection,
-                        BrowserContract.Bookmarks.IS_FOLDER + " != 0",
+                        where,
                         null,
                         null);
             default:
@@ -452,6 +457,7 @@
         }
     }
 
+    @Override
     protected void onCreate(Bundle icicle) {
         super.onCreate(icicle);
         if (DEBUG_CRASH) {