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

Unified Diff: chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm

Issue 8759017: BookmarkModel cleanup. synced_node is now mobile_node and I'm nuking (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to trunk fix sync_integration_tests and extension test Created 9 years, 1 month 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/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm b/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm
index e2b50d35026a074a9d1f29ce4f6b0473a4d8315c..b4739d223bbf39a63817f0a208048ad4ec9d8d09 100644
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller_unittest.mm
@@ -175,11 +175,7 @@ TEST_F(BookmarkBubbleControllerTest, TestFillInFolder) {
// Verify that the top level folders are displayed correctly.
EXPECT_TRUE([titles containsObject:@"Other Bookmarks"]);
EXPECT_TRUE([titles containsObject:@"Bookmarks Bar"]);
- if (model->synced_node()->IsVisible()) {
- EXPECT_TRUE([titles containsObject:@"Synced Bookmarks"]);
- } else {
- EXPECT_FALSE([titles containsObject:@"Synced Bookmarks"]);
- }
+ EXPECT_TRUE([titles containsObject:@"Mobile Bookmarks"]);
}
// Confirm ability to handle folders with blank name.

Powered by Google App Engine
This is Rietveld 408576698