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

Side by Side Diff: chrome/browser/ui/cocoa/bookmarks/bookmark_menu_bridge_unittest.mm

Issue 2809003002: Making bookmark folder context menu display the number of bookmarks that will be opened by Open All (Closed)
Patch Set: All changes to add count to context menu Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import <AppKit/AppKit.h> 5 #import <AppKit/AppKit.h>
6 6
7 #include "base/strings/string16.h" 7 #include "base/strings/string16.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/app/chrome_command_ids.h" 10 #include "chrome/app/chrome_command_ids.h"
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 EXPECT_EQ(0, [menu numberOfItems]); 253 EXPECT_EQ(0, [menu numberOfItems]);
254 254
255 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL, 255 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL,
256 IDS_BOOKMARK_BAR_OPEN_ALL, root, menu, true); 256 IDS_BOOKMARK_BAR_OPEN_ALL, root, menu, true);
257 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, 257 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW,
258 IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, root, menu, true); 258 IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, root, menu, true);
259 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_INCOGNITO, 259 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_INCOGNITO,
260 IDS_BOOKMARK_BAR_OPEN_INCOGNITO, root, menu, true); 260 IDS_BOOKMARK_BAR_OPEN_INCOGNITO, root, menu, true);
261 EXPECT_EQ(3, [menu numberOfItems]); 261 EXPECT_EQ(3, [menu numberOfItems]);
262 262
263 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_ALL); 263 title = l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_ALL, 0);
264 item = [menu itemWithTitle:title]; 264 item = [menu itemWithTitle:title];
265 EXPECT_TRUE(item); 265 EXPECT_TRUE(item);
266 EXPECT_EQ(@selector(openAllBookmarks:), [item action]); 266 EXPECT_EQ(@selector(openAllBookmarks:), [item action]);
267 EXPECT_TRUE([item isEnabled]); 267 EXPECT_TRUE([item isEnabled]);
268 268
269 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW); 269 title =
270 l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, 0);
270 item = [menu itemWithTitle:title]; 271 item = [menu itemWithTitle:title];
271 EXPECT_TRUE(item); 272 EXPECT_TRUE(item);
272 EXPECT_EQ(@selector(openAllBookmarksNewWindow:), [item action]); 273 EXPECT_EQ(@selector(openAllBookmarksNewWindow:), [item action]);
273 EXPECT_TRUE([item isEnabled]); 274 EXPECT_TRUE([item isEnabled]);
274 275
275 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_INCOGNITO); 276 title = l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_INCOGNITO, 0);
276 item = [menu itemWithTitle:title]; 277 item = [menu itemWithTitle:title];
277 EXPECT_TRUE(item); 278 EXPECT_TRUE(item);
278 EXPECT_EQ(@selector(openAllBookmarksIncognitoWindow:), [item action]); 279 EXPECT_EQ(@selector(openAllBookmarksIncognitoWindow:), [item action]);
279 EXPECT_TRUE([item isEnabled]); 280 EXPECT_TRUE([item isEnabled]);
280 281
281 ClearBookmarkMenu(bridge_.get(), menu); 282 ClearBookmarkMenu(bridge_.get(), menu);
282 EXPECT_EQ(0, [menu numberOfItems]); 283 EXPECT_EQ(0, [menu numberOfItems]);
283 284
284 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL, 285 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL,
285 IDS_BOOKMARK_BAR_OPEN_ALL, root, menu, false); 286 IDS_BOOKMARK_BAR_OPEN_ALL, root, menu, false);
286 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, 287 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW,
287 IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, root, menu, false); 288 IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, root, menu, false);
288 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_INCOGNITO, 289 AddItemToMenu(bridge_.get(), IDC_BOOKMARK_BAR_OPEN_ALL_INCOGNITO,
289 IDS_BOOKMARK_BAR_OPEN_INCOGNITO, root, menu, false); 290 IDS_BOOKMARK_BAR_OPEN_INCOGNITO, root, menu, false);
290 EXPECT_EQ(3, [menu numberOfItems]); 291 EXPECT_EQ(3, [menu numberOfItems]);
291 292
292 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_ALL); 293 title = l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_ALL, 0);
293 item = [menu itemWithTitle:title]; 294 item = [menu itemWithTitle:title];
294 EXPECT_TRUE(item); 295 EXPECT_TRUE(item);
295 EXPECT_EQ(nil, [item action]); 296 EXPECT_EQ(nil, [item action]);
296 EXPECT_FALSE([item isEnabled]); 297 EXPECT_FALSE([item isEnabled]);
297 298
298 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW); 299 title =
300 l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_ALL_NEW_WINDOW, 0);
299 item = [menu itemWithTitle:title]; 301 item = [menu itemWithTitle:title];
300 EXPECT_TRUE(item); 302 EXPECT_TRUE(item);
301 EXPECT_EQ(nil, [item action]); 303 EXPECT_EQ(nil, [item action]);
302 EXPECT_FALSE([item isEnabled]); 304 EXPECT_FALSE([item isEnabled]);
303 305
304 title = l10n_util::GetNSStringWithFixup(IDS_BOOKMARK_BAR_OPEN_INCOGNITO); 306 title = l10n_util::GetPluralNSStringF(IDS_BOOKMARK_BAR_OPEN_INCOGNITO, 0);
305 item = [menu itemWithTitle:title]; 307 item = [menu itemWithTitle:title];
306 EXPECT_TRUE(item); 308 EXPECT_TRUE(item);
307 EXPECT_EQ(nil, [item action]); 309 EXPECT_EQ(nil, [item action]);
308 EXPECT_FALSE([item isEnabled]); 310 EXPECT_FALSE([item isEnabled]);
309 } 311 }
310 312
311 // Makes sure our internal map of BookmarkNode to NSMenuItem works. 313 // Makes sure our internal map of BookmarkNode to NSMenuItem works.
312 TEST_F(BookmarkMenuBridgeTest, TestGetMenuItemForNode) { 314 TEST_F(BookmarkMenuBridgeTest, TestGetMenuItemForNode) {
313 base::string16 empty; 315 base::string16 empty;
314 NSMenu* menu = bridge_->menu_; 316 NSMenu* menu = bridge_->menu_;
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
392 EXPECT_TRUE([item image]); 394 EXPECT_TRUE([item image]);
393 395
394 model->SetTitle(node, ASCIIToUTF16("New Title")); 396 model->SetTitle(node, ASCIIToUTF16("New Title"));
395 397
396 item = [menu itemWithTitle:@"Test Item"]; 398 item = [menu itemWithTitle:@"Test Item"];
397 EXPECT_FALSE(item); 399 EXPECT_FALSE(item);
398 item = [menu itemWithTitle:@"New Title"]; 400 item = [menu itemWithTitle:@"New Title"];
399 EXPECT_TRUE(item); 401 EXPECT_TRUE(item);
400 } 402 }
401 403
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698