| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.h" | 5 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_app.h" |
| 6 | 6 |
| 7 #include <algorithm> | 7 #include <algorithm> |
| 8 #include <string> | 8 #include <string> |
| 9 #include <vector> | 9 #include <vector> |
| 10 | 10 |
| 11 #include "ash/launcher/launcher_model.h" | 11 #include "ash/launcher/launcher_model.h" |
| 12 #include "base/command_line.h" | 12 #include "base/command_line.h" |
| 13 #include "base/compiler_specific.h" | 13 #include "base/compiler_specific.h" |
| 14 #include "base/file_path.h" | 14 #include "base/file_path.h" |
| 15 #include "base/memory/scoped_ptr.h" | 15 #include "base/memory/scoped_ptr.h" |
| 16 #include "base/message_loop.h" | 16 #include "base/message_loop.h" |
| 17 #include "base/utf_string_conversions.h" | 17 #include "base/utf_string_conversions.h" |
| 18 #include "base/values.h" | 18 #include "base/values.h" |
| 19 #include "chrome/browser/extensions/extension_service.h" | 19 #include "chrome/browser/extensions/extension_service.h" |
| 20 #include "chrome/browser/extensions/test_extension_system.h" | 20 #include "chrome/browser/extensions/test_extension_system.h" |
| 21 #include "chrome/browser/ui/ash/chrome_launcher_prefs.h" | 21 #include "chrome/browser/ui/ash/chrome_launcher_prefs.h" |
| 22 #include "chrome/browser/ui/ash/launcher/launcher_item_controller.h" | 22 #include "chrome/browser/ui/ash/launcher/launcher_item_controller.h" |
| 23 #include "chrome/browser/ui/browser_commands.h" | 23 #include "chrome/browser/ui/browser_commands.h" |
| 24 #include "chrome/browser/ui/browser_finder.h" |
| 24 #include "chrome/browser/ui/browser_list.h" | 25 #include "chrome/browser/ui/browser_list.h" |
| 25 #include "chrome/browser/ui/tabs/tab_strip_model.h" | 26 #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| 26 #include "chrome/common/extensions/extension.h" | 27 #include "chrome/common/extensions/extension.h" |
| 27 #include "chrome/common/pref_names.h" | 28 #include "chrome/common/pref_names.h" |
| 28 #include "chrome/test/base/browser_with_test_window_test.h" | 29 #include "chrome/test/base/browser_with_test_window_test.h" |
| 29 #include "chrome/test/base/testing_pref_service_syncable.h" | 30 #include "chrome/test/base/testing_pref_service_syncable.h" |
| 30 #include "chrome/test/base/testing_profile.h" | 31 #include "chrome/test/base/testing_profile.h" |
| 31 #include "content/public/test/test_browser_thread.h" | 32 #include "content/public/test/test_browser_thread.h" |
| 32 #include "testing/gtest/include/gtest/gtest.h" | 33 #include "testing/gtest/include/gtest/gtest.h" |
| 33 #include "ui/base/models/menu_model.h" | 34 #include "ui/base/models/menu_model.h" |
| (...skipping 276 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 310 EXPECT_EQ(expected_menu_items, menu->GetItemCount()); | 311 EXPECT_EQ(expected_menu_items, menu->GetItemCount()); |
| 311 EXPECT_FALSE(menu->IsEnabledAt(first_item)); | 312 EXPECT_FALSE(menu->IsEnabledAt(first_item)); |
| 312 if (expected_items) { | 313 if (expected_items) { |
| 313 EXPECT_EQ(ui::MenuModel::TYPE_SEPARATOR, | 314 EXPECT_EQ(ui::MenuModel::TYPE_SEPARATOR, |
| 314 menu->GetTypeAt(first_item + 1)); | 315 menu->GetTypeAt(first_item + 1)); |
| 315 } | 316 } |
| 316 } | 317 } |
| 317 | 318 |
| 318 // Check that browsers get reflected correctly in the launcher menu. | 319 // Check that browsers get reflected correctly in the launcher menu. |
| 319 TEST_F(ChromeLauncherControllerPerAppTest, BrowserMenuGeneration) { | 320 TEST_F(ChromeLauncherControllerPerAppTest, BrowserMenuGeneration) { |
| 320 EXPECT_EQ(1U, BrowserList::size()); | 321 EXPECT_EQ(1U, chrome::GetTotalBrowserCount()); |
| 321 chrome::NewTab(browser()); | 322 chrome::NewTab(browser()); |
| 322 | 323 |
| 323 ChromeLauncherControllerPerApp launcher_controller(profile(), &model_); | 324 ChromeLauncherControllerPerApp launcher_controller(profile(), &model_); |
| 324 launcher_controller.Init(); | 325 launcher_controller.Init(); |
| 325 | 326 |
| 326 // Check that the browser list is empty at this time. | 327 // Check that the browser list is empty at this time. |
| 327 ash::LauncherItem item_browser; | 328 ash::LauncherItem item_browser; |
| 328 item_browser.type = ash::TYPE_BROWSER_SHORTCUT; | 329 item_browser.type = ash::TYPE_BROWSER_SHORTCUT; |
| 329 CheckMenuCreation(&launcher_controller, item_browser, 0, NULL, true); | 330 CheckMenuCreation(&launcher_controller, item_browser, 0, NULL, true); |
| 330 | 331 |
| (...skipping 25 matching lines...) Expand all Loading... |
| 356 | 357 |
| 357 // Apparently we have to close all tabs we have. | 358 // Apparently we have to close all tabs we have. |
| 358 chrome::CloseTab(browser2.get()); | 359 chrome::CloseTab(browser2.get()); |
| 359 } | 360 } |
| 360 | 361 |
| 361 // Check that V1 apps are correctly reflected in the launcher menu using the | 362 // Check that V1 apps are correctly reflected in the launcher menu using the |
| 362 // refocus logic. | 363 // refocus logic. |
| 363 // Note that the extension matching logic is tested by the extension system | 364 // Note that the extension matching logic is tested by the extension system |
| 364 // and does not need a separate test here. | 365 // and does not need a separate test here. |
| 365 TEST_F(ChromeLauncherControllerPerAppTest, V1AppMenuGeneration) { | 366 TEST_F(ChromeLauncherControllerPerAppTest, V1AppMenuGeneration) { |
| 366 EXPECT_EQ(1U, BrowserList::size()); | 367 EXPECT_EQ(1U, chrome::GetTotalBrowserCount()); |
| 367 EXPECT_EQ(0, browser()->tab_strip_model()->count()); | 368 EXPECT_EQ(0, browser()->tab_strip_model()->count()); |
| 368 chrome::NewTab(browser()); | 369 chrome::NewTab(browser()); |
| 369 BrowserList::SetLastActive(browser()); | 370 BrowserList::SetLastActive(browser()); |
| 370 | 371 |
| 371 ChromeLauncherControllerPerApp launcher_controller(profile(), &model_); | 372 ChromeLauncherControllerPerApp launcher_controller(profile(), &model_); |
| 372 launcher_controller.Init(); | 373 launcher_controller.Init(); |
| 373 | 374 |
| 374 // Model should only contain the browser shortcut and app list items. | 375 // Model should only contain the browser shortcut and app list items. |
| 375 EXPECT_EQ(2, model_.item_count()); | 376 EXPECT_EQ(2, model_.item_count()); |
| 376 EXPECT_FALSE(launcher_controller.IsAppPinned(extension3_->id())); | 377 EXPECT_FALSE(launcher_controller.IsAppPinned(extension3_->id())); |
| (...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 481 { | 482 { |
| 482 scoped_ptr<ui::MenuModel> menu( | 483 scoped_ptr<ui::MenuModel> menu( |
| 483 launcher_controller.CreateApplicationMenu(item_gmail)); | 484 launcher_controller.CreateApplicationMenu(item_gmail)); |
| 484 int first_item = | 485 int first_item = |
| 485 (menu->GetTypeAt(0) == ui::MenuModel::TYPE_SEPARATOR) ? 1 : 0; | 486 (menu->GetTypeAt(0) == ui::MenuModel::TYPE_SEPARATOR) ? 1 : 0; |
| 486 menu->ActivatedAt(first_item + 3); | 487 menu->ActivatedAt(first_item + 3); |
| 487 } | 488 } |
| 488 // Now the active tab should be the second item. | 489 // Now the active tab should be the second item. |
| 489 EXPECT_EQ(0, browser()->tab_strip_model()->active_index()); | 490 EXPECT_EQ(0, browser()->tab_strip_model()->active_index()); |
| 490 } | 491 } |
| OLD | NEW |