Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(231)

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java

Issue 1935413003: [Android Bookmark] Remove all bookmark section from bookmark manager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make a histogram obsolete Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java
index 39660e14ad5a2fd75458fdccc0f97d5834a44e2e..75b2b83b68f9f85e19867c54a38e49604db4df52 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/bookmarks/BookmarkModelTest.java
@@ -20,7 +20,6 @@ import java.util.ArrayList;
import java.util.Arrays;
import java.util.HashSet;
import java.util.List;
-import java.util.Stack;
import java.util.concurrent.Semaphore;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicReference;
@@ -63,37 +62,6 @@ public class BookmarkModelTest extends NativeLibraryTestBase {
@UiThreadTest
@SmallTest
@Feature({"Bookmark"})
- public void testGetAllBookmarkIDsOrderedByCreationDate() throws InterruptedException {
- BookmarkId folderA = mBookmarkModel.addFolder(mMobileNode, 0, "a");
- BookmarkId folderB = mBookmarkModel.addFolder(mDesktopNode, 0, "b");
-
- Stack<BookmarkId> stack = new Stack<BookmarkId>();
- stack.push(addBookmark(folderA, 0, "a", "http://www.medium.com"));
- // If add bookmarks too fast, eventually some bookmarks will have the same timestamp, which
- // confuses the bookmark model.
- Thread.sleep(20);
- stack.push(addBookmark(folderB, 0, "b", "http://aurimas.com"));
- Thread.sleep(20);
- stack.push(addBookmark(mMobileNode, 0, "c", "http://www.aurimas.com"));
- Thread.sleep(20);
- stack.push(addBookmark(mDesktopNode, 0, "d", "http://www.aurimas.org"));
- Thread.sleep(20);
- stack.push(addBookmark(mOtherNode, 0, "e", "http://www.google.com"));
- Thread.sleep(20);
- stack.push(addBookmark(folderA, 0, "f", "http://www.newt.com"));
- Thread.sleep(20);
- stack.push(addBookmark(folderB, 0, "g", "http://kkimlabs.com"));
-
- List<BookmarkId> bookmarks = mBookmarkModel.getAllBookmarkIDsOrderedByCreationDate();
- assertEquals(stack.size(), bookmarks.size());
- for (BookmarkId returnedBookmark : bookmarks) {
- assertEquals(stack.pop(), returnedBookmark);
- }
- }
-
- @UiThreadTest
- @SmallTest
- @Feature({"Bookmark"})
public void testBookmarkPropertySetters() {
BookmarkId folderA = mBookmarkModel.addFolder(mMobileNode, 0, "a");

Powered by Google App Engine
This is Rietveld 408576698