| OLD | NEW |
| 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 Loading... |
| 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 Loading... |
| 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 |
| OLD | NEW |