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

Unified Diff: chrome/browser/ui/browser_browsertest.cc

Issue 10700130: Introduce LaunchParams struct for opening chrome apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix bad merge Created 8 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/browser_browsertest.cc
diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
index 776e800227a27a1886d9016d4f8378f1f4b21295..7763f141ad12145007a238ed387e58f394237518 100644
--- a/chrome/browser/ui/browser_browsertest.cc
+++ b/chrome/browser/ui/browser_browsertest.cc
@@ -1000,13 +1000,10 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, OpenAppWindowLikeNtp) {
const Extension* extension_app = GetExtension();
// Launch it in a window, as AppLauncherHandler::HandleLaunchApp() would.
- WebContents* app_window =
- application_launch::OpenApplication(browser()->profile(),
- extension_app,
- extension_misc::LAUNCH_WINDOW,
- GURL(),
- NEW_WINDOW,
- NULL);
+ WebContents* app_window = application_launch::OpenApplication(
+ application_launch::LaunchParams(browser()->profile(), extension_app,
+ extension_misc::LAUNCH_WINDOW,
+ NEW_WINDOW));
ASSERT_TRUE(app_window);
// Apps launched in a window from the NTP do not have extension_app set in
« no previous file with comments | « chrome/browser/extensions/platform_app_browsertest_util.cc ('k') | chrome/browser/ui/extensions/application_launch.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698