| Index: chrome/android/javatests_shell/src/org/chromium/chrome/browser/enhanced_bookmarks/EnhancedBookmarksModelTest.java
|
| diff --git a/chrome/android/javatests_shell/src/org/chromium/chrome/browser/enhanced_bookmarks/EnhancedBookmarksModelTest.java b/chrome/android/javatests_shell/src/org/chromium/chrome/browser/enhanced_bookmarks/EnhancedBookmarksModelTest.java
|
| index d5a8fdc356125ab62eb0e49279954ce081340b9c..56efe4d285c340e6aaccc4bf16a4181be72b1488 100644
|
| --- a/chrome/android/javatests_shell/src/org/chromium/chrome/browser/enhanced_bookmarks/EnhancedBookmarksModelTest.java
|
| +++ b/chrome/android/javatests_shell/src/org/chromium/chrome/browser/enhanced_bookmarks/EnhancedBookmarksModelTest.java
|
| @@ -100,23 +100,19 @@ public class EnhancedBookmarksModelTest extends ChromeShellTestBase{
|
|
|
| mBookmarksModel.setBookmarkTitle(bookmarkA, "auri");
|
| mBookmarksModel.setBookmarkUrl(bookmarkA, "http://auri.org/");
|
| - mBookmarksModel.setBookmarkDescription(bookmarkA, "auri");
|
| - verifyBookmark(bookmarkA, "auri", "http://auri.org/", false, mDesktopNode, "auri");
|
| + verifyBookmark(bookmarkA, "auri", "http://auri.org/", false, mDesktopNode);
|
|
|
| mBookmarksModel.setBookmarkTitle(bookmarkB, "lauri");
|
| mBookmarksModel.setBookmarkUrl(bookmarkB, "http://lauri.org/");
|
| - mBookmarksModel.setBookmarkDescription(bookmarkB, "lauri");
|
| - verifyBookmark(bookmarkB, "lauri", "http://lauri.org/", false, mMobileNode, "lauri");
|
| + verifyBookmark(bookmarkB, "lauri", "http://lauri.org/", false, mMobileNode);
|
|
|
| mBookmarksModel.setBookmarkTitle(bookmarkC, "mauri");
|
| mBookmarksModel.setBookmarkUrl(bookmarkC, "http://mauri.org/");
|
| - mBookmarksModel.setBookmarkDescription(bookmarkC, "mauri");
|
| - verifyBookmark(bookmarkC, "mauri", "http://mauri.org/", false, mOtherNode, "mauri");
|
| + verifyBookmark(bookmarkC, "mauri", "http://mauri.org/", false, mOtherNode);
|
|
|
| mBookmarksModel.setBookmarkTitle(bookmarkD, "kauri");
|
| mBookmarksModel.setBookmarkUrl(bookmarkD, "http://kauri.org/");
|
| - mBookmarksModel.setBookmarkDescription(bookmarkD, "kauri");
|
| - verifyBookmark(bookmarkD, "kauri", "http://kauri.org/", false, folderA, "kauri");
|
| + verifyBookmark(bookmarkD, "kauri", "http://kauri.org/", false, folderA);
|
| }
|
|
|
| @SuppressFBWarnings("DLS_DEAD_LOCAL_STORE")
|
| @@ -177,40 +173,38 @@ public class EnhancedBookmarksModelTest extends ChromeShellTestBase{
|
| @Feature({"Bookmark"})
|
| public void testAddBookmarksAndFolders() {
|
| BookmarkId bookmarkA = mBookmarksModel.addBookmark(mDesktopNode, 0, "a", "http://a.com");
|
| - verifyBookmark(bookmarkA, "a", "http://a.com/", false, mDesktopNode, "");
|
| + verifyBookmark(bookmarkA, "a", "http://a.com/", false, mDesktopNode);
|
|
|
| BookmarkId bookmarkB = mBookmarksModel.addBookmark(mOtherNode, 0, "b", "http://b.com");
|
| - verifyBookmark(bookmarkB, "b", "http://b.com/", false, mOtherNode, "");
|
| + verifyBookmark(bookmarkB, "b", "http://b.com/", false, mOtherNode);
|
|
|
| BookmarkId bookmarkC = mBookmarksModel.addBookmark(mMobileNode, 0, "c", "http://c.com");
|
| - verifyBookmark(bookmarkC, "c", "http://c.com/", false, mMobileNode, "");
|
| + verifyBookmark(bookmarkC, "c", "http://c.com/", false, mMobileNode);
|
|
|
| BookmarkId folderA = mBookmarksModel.addFolder(mOtherNode, 0, "fa");
|
| - verifyBookmark(folderA, "fa", null, true, mOtherNode, "");
|
| + verifyBookmark(folderA, "fa", null, true, mOtherNode);
|
|
|
| BookmarkId folderB = mBookmarksModel.addFolder(mDesktopNode, 0, "fb");
|
| - verifyBookmark(folderB, "fb", null, true, mDesktopNode, "");
|
| + verifyBookmark(folderB, "fb", null, true, mDesktopNode);
|
|
|
| BookmarkId folderC = mBookmarksModel.addFolder(mMobileNode, 0, "fc");
|
| - verifyBookmark(folderC, "fc", null, true, mMobileNode, "");
|
| + verifyBookmark(folderC, "fc", null, true, mMobileNode);
|
|
|
| BookmarkId bookmarkAA = mBookmarksModel.addBookmark(folderA, 0, "aa", "http://aa.com");
|
| - verifyBookmark(bookmarkAA, "aa", "http://aa.com/", false, folderA, "");
|
| + verifyBookmark(bookmarkAA, "aa", "http://aa.com/", false, folderA);
|
|
|
| BookmarkId folderAA = mBookmarksModel.addFolder(folderA, 0, "faa");
|
| - verifyBookmark(folderAA, "faa", null, true, folderA, "");
|
| + verifyBookmark(folderAA, "faa", null, true, folderA);
|
| }
|
|
|
| private void verifyBookmark(BookmarkId idToVerify, String expectedTitle,
|
| - String expectedUrl, boolean isFolder, BookmarkId expectedParent,
|
| - String expectedDescription) {
|
| + String expectedUrl, boolean isFolder, BookmarkId expectedParent) {
|
| assertNotNull(idToVerify);
|
| BookmarkItem item = mBookmarksModel.getBookmarkById(idToVerify);
|
| assertEquals(expectedTitle, item.getTitle());
|
| assertEquals(isFolder, item.isFolder());
|
| if (!isFolder) assertEquals(expectedUrl, item.getUrl());
|
| assertEquals(expectedParent, item.getParentId());
|
| - assertEquals(expectedDescription, mBookmarksModel.getBookmarkDescription(idToVerify));
|
| }
|
|
|
| /**
|
|
|