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

Side by Side Diff: chrome/browser/ui/browser_browsertest.cc

Issue 720633003: Revert of CL 722703002 due to build breakage (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: revert Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 <string> 5 #include <string>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 22 matching lines...) Expand all
33 #include "chrome/browser/translate/cld_data_harness_factory.h" 33 #include "chrome/browser/translate/cld_data_harness_factory.h"
34 #include "chrome/browser/ui/browser.h" 34 #include "chrome/browser/ui/browser.h"
35 #include "chrome/browser/ui/browser_command_controller.h" 35 #include "chrome/browser/ui/browser_command_controller.h"
36 #include "chrome/browser/ui/browser_commands.h" 36 #include "chrome/browser/ui/browser_commands.h"
37 #include "chrome/browser/ui/browser_finder.h" 37 #include "chrome/browser/ui/browser_finder.h"
38 #include "chrome/browser/ui/browser_iterator.h" 38 #include "chrome/browser/ui/browser_iterator.h"
39 #include "chrome/browser/ui/browser_navigator.h" 39 #include "chrome/browser/ui/browser_navigator.h"
40 #include "chrome/browser/ui/browser_tabstrip.h" 40 #include "chrome/browser/ui/browser_tabstrip.h"
41 #include "chrome/browser/ui/browser_ui_prefs.h" 41 #include "chrome/browser/ui/browser_ui_prefs.h"
42 #include "chrome/browser/ui/browser_window.h" 42 #include "chrome/browser/ui/browser_window.h"
43 #include "chrome/browser/ui/extensions/app_launch_params.h"
44 #include "chrome/browser/ui/extensions/application_launch.h" 43 #include "chrome/browser/ui/extensions/application_launch.h"
45 #include "chrome/browser/ui/host_desktop.h" 44 #include "chrome/browser/ui/host_desktop.h"
46 #include "chrome/browser/ui/startup/startup_browser_creator.h" 45 #include "chrome/browser/ui/startup/startup_browser_creator.h"
47 #include "chrome/browser/ui/startup/startup_browser_creator_impl.h" 46 #include "chrome/browser/ui/startup/startup_browser_creator_impl.h"
48 #include "chrome/browser/ui/tabs/pinned_tab_codec.h" 47 #include "chrome/browser/ui/tabs/pinned_tab_codec.h"
49 #include "chrome/browser/ui/tabs/tab_strip_model.h" 48 #include "chrome/browser/ui/tabs/tab_strip_model.h"
50 #include "chrome/common/chrome_switches.h" 49 #include "chrome/common/chrome_switches.h"
51 #include "chrome/common/extensions/manifest_handlers/app_launch_info.h" 50 #include "chrome/common/extensions/manifest_handlers/app_launch_info.h"
52 #include "chrome/common/pref_names.h" 51 #include "chrome/common/pref_names.h"
53 #include "chrome/common/url_constants.h" 52 #include "chrome/common/url_constants.h"
(...skipping 24 matching lines...) Expand all
78 #include "content/public/browser/web_contents.h" 77 #include "content/public/browser/web_contents.h"
79 #include "content/public/browser/web_contents_observer.h" 78 #include "content/public/browser/web_contents_observer.h"
80 #include "content/public/common/frame_navigate_params.h" 79 #include "content/public/common/frame_navigate_params.h"
81 #include "content/public/common/renderer_preferences.h" 80 #include "content/public/common/renderer_preferences.h"
82 #include "content/public/common/url_constants.h" 81 #include "content/public/common/url_constants.h"
83 #include "content/public/test/browser_test_utils.h" 82 #include "content/public/test/browser_test_utils.h"
84 #include "content/public/test/test_navigation_observer.h" 83 #include "content/public/test/test_navigation_observer.h"
85 #include "extensions/browser/extension_registry.h" 84 #include "extensions/browser/extension_registry.h"
86 #include "extensions/browser/extension_system.h" 85 #include "extensions/browser/extension_system.h"
87 #include "extensions/browser/uninstall_reason.h" 86 #include "extensions/browser/uninstall_reason.h"
88 #include "extensions/common/constants.h"
89 #include "extensions/common/extension.h" 87 #include "extensions/common/extension.h"
90 #include "extensions/common/extension_set.h" 88 #include "extensions/common/extension_set.h"
91 #include "net/dns/mock_host_resolver.h" 89 #include "net/dns/mock_host_resolver.h"
92 #include "net/test/spawned_test_server/spawned_test_server.h" 90 #include "net/test/spawned_test_server/spawned_test_server.h"
93 #include "ui/base/l10n/l10n_util.h" 91 #include "ui/base/l10n/l10n_util.h"
94 #include "ui/base/page_transition_types.h" 92 #include "ui/base/page_transition_types.h"
95 93
96 #if defined(OS_MACOSX) 94 #if defined(OS_MACOSX)
97 #include "base/mac/mac_util.h" 95 #include "base/mac/mac_util.h"
98 #include "base/mac/scoped_nsautorelease_pool.h" 96 #include "base/mac/scoped_nsautorelease_pool.h"
(...skipping 1238 matching lines...) Expand 10 before | Expand all | Expand 10 after
1337 // bar settings are correct. 1335 // bar settings are correct.
1338 IN_PROC_BROWSER_TEST_F(BrowserTest, ShouldShowLocationBar) { 1336 IN_PROC_BROWSER_TEST_F(BrowserTest, ShouldShowLocationBar) {
1339 ASSERT_TRUE(test_server()->Start()); 1337 ASSERT_TRUE(test_server()->Start());
1340 1338
1341 // Load an app. 1339 // Load an app.
1342 host_resolver()->AddRule("www.example.com", "127.0.0.1"); 1340 host_resolver()->AddRule("www.example.com", "127.0.0.1");
1343 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app/"))); 1341 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app/")));
1344 const Extension* extension_app = GetExtension(); 1342 const Extension* extension_app = GetExtension();
1345 1343
1346 // Launch it in a window, as AppLauncherHandler::HandleLaunchApp() would. 1344 // Launch it in a window, as AppLauncherHandler::HandleLaunchApp() would.
1347 WebContents* app_window = OpenApplication(AppLaunchParams( 1345 WebContents* app_window =
1348 browser()->profile(), extension_app, extensions::LAUNCH_CONTAINER_WINDOW, 1346 OpenApplication(AppLaunchParams(browser()->profile(),
1349 NEW_WINDOW, extensions::SOURCE_UNTRACKED)); 1347 extension_app,
1348 extensions::LAUNCH_CONTAINER_WINDOW,
1349 NEW_WINDOW));
1350 ASSERT_TRUE(app_window); 1350 ASSERT_TRUE(app_window);
1351 1351
1352 DevToolsWindow* devtools_window = 1352 DevToolsWindow* devtools_window =
1353 DevToolsWindowTesting::OpenDevToolsWindowSync(browser(), false); 1353 DevToolsWindowTesting::OpenDevToolsWindowSync(browser(), false);
1354 1354
1355 // The launch should have created a new app browser and a dev tools browser. 1355 // The launch should have created a new app browser and a dev tools browser.
1356 ASSERT_EQ(3u, 1356 ASSERT_EQ(3u,
1357 chrome::GetBrowserCount(browser()->profile(), 1357 chrome::GetBrowserCount(browser()->profile(),
1358 browser()->host_desktop_type())); 1358 browser()->host_desktop_type()));
1359 1359
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
1529 #if !defined(OS_MACOSX) 1529 #if !defined(OS_MACOSX)
1530 IN_PROC_BROWSER_TEST_F(BrowserTest, OpenAppWindowLikeNtp) { 1530 IN_PROC_BROWSER_TEST_F(BrowserTest, OpenAppWindowLikeNtp) {
1531 ASSERT_TRUE(test_server()->Start()); 1531 ASSERT_TRUE(test_server()->Start());
1532 1532
1533 // Load an app 1533 // Load an app
1534 host_resolver()->AddRule("www.example.com", "127.0.0.1"); 1534 host_resolver()->AddRule("www.example.com", "127.0.0.1");
1535 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app/"))); 1535 ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app/")));
1536 const Extension* extension_app = GetExtension(); 1536 const Extension* extension_app = GetExtension();
1537 1537
1538 // Launch it in a window, as AppLauncherHandler::HandleLaunchApp() would. 1538 // Launch it in a window, as AppLauncherHandler::HandleLaunchApp() would.
1539 WebContents* app_window = OpenApplication(AppLaunchParams( 1539 WebContents* app_window = OpenApplication(
1540 browser()->profile(), extension_app, extensions::LAUNCH_CONTAINER_WINDOW, 1540 AppLaunchParams(browser()->profile(), extension_app,
1541 NEW_WINDOW, extensions::SOURCE_UNTRACKED)); 1541 extensions::LAUNCH_CONTAINER_WINDOW, NEW_WINDOW));
1542 ASSERT_TRUE(app_window); 1542 ASSERT_TRUE(app_window);
1543 1543
1544 // Apps launched in a window from the NTP have an extensions tab helper but 1544 // Apps launched in a window from the NTP have an extensions tab helper but
1545 // do not have extension_app set in it. 1545 // do not have extension_app set in it.
1546 ASSERT_TRUE(extensions::TabHelper::FromWebContents(app_window)); 1546 ASSERT_TRUE(extensions::TabHelper::FromWebContents(app_window));
1547 EXPECT_FALSE( 1547 EXPECT_FALSE(
1548 extensions::TabHelper::FromWebContents(app_window)->extension_app()); 1548 extensions::TabHelper::FromWebContents(app_window)->extension_app());
1549 EXPECT_EQ(extensions::AppLaunchInfo::GetFullLaunchURL(extension_app), 1549 EXPECT_EQ(extensions::AppLaunchInfo::GetFullLaunchURL(extension_app),
1550 app_window->GetURL()); 1550 app_window->GetURL());
1551 1551
(...skipping 1102 matching lines...) Expand 10 before | Expand all | Expand 10 after
2654 #endif 2654 #endif
2655 EXPECT_EQ(exp_commit_size, rwhv_commit_size2); 2655 EXPECT_EQ(exp_commit_size, rwhv_commit_size2);
2656 EXPECT_EQ(exp_commit_size, wcv_commit_size2); 2656 EXPECT_EQ(exp_commit_size, wcv_commit_size2);
2657 gfx::Size exp_final_size(initial_wcv_size); 2657 gfx::Size exp_final_size(initial_wcv_size);
2658 exp_final_size.Enlarge(wcv_resize_insets.width(), 2658 exp_final_size.Enlarge(wcv_resize_insets.width(),
2659 wcv_resize_insets.height() + height_inset); 2659 wcv_resize_insets.height() + height_inset);
2660 EXPECT_EQ(exp_final_size, 2660 EXPECT_EQ(exp_final_size,
2661 web_contents->GetRenderWidgetHostView()->GetViewBounds().size()); 2661 web_contents->GetRenderWidgetHostView()->GetViewBounds().size());
2662 EXPECT_EQ(exp_final_size, web_contents->GetContainerBounds().size()); 2662 EXPECT_EQ(exp_final_size, web_contents->GetContainerBounds().size());
2663 } 2663 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/ash/launcher/chrome_launcher_controller_browsertest.cc ('k') | chrome/browser/ui/chrome_pages.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698