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

Side by Side Diff: chrome/browser/ui/ash/accelerator_commands_browsertest.cc

Issue 187443008: Elim Browser::AppType (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "ash/accelerators/accelerator_commands.h" 5 #include "ash/accelerators/accelerator_commands.h"
6 6
7 #include "apps/app_window.h" 7 #include "apps/app_window.h"
8 #include "apps/ui/native_app_window.h" 8 #include "apps/ui/native_app_window.h"
9 #include "ash/ash_switches.h" 9 #include "ash/ash_switches.h"
10 #include "ash/shell.h" 10 #include "ash/shell.h"
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
178 ash::accelerators::ToggleFullscreen(); 178 ash::accelerators::ToggleFullscreen();
179 EXPECT_TRUE(IsInitialShowState(window_state)); 179 EXPECT_TRUE(IsInitialShowState(window_state));
180 180
181 // 2) ToggleFullscreen() should have no effect on windows which cannot be 181 // 2) ToggleFullscreen() should have no effect on windows which cannot be
182 // maximized. 182 // maximized.
183 window_state->window()->SetProperty(aura::client::kCanMaximizeKey, false); 183 window_state->window()->SetProperty(aura::client::kCanMaximizeKey, false);
184 ash::accelerators::ToggleFullscreen(); 184 ash::accelerators::ToggleFullscreen();
185 EXPECT_TRUE(IsInitialShowState(window_state)); 185 EXPECT_TRUE(IsInitialShowState(window_state));
186 186
187 // 3) Hosted apps. 187 // 3) Hosted apps.
188 Browser::CreateParams browser_create_params(Browser::TYPE_POPUP, 188 Browser::CreateParams browser_create_params(
189 browser()->profile(), chrome::HOST_DESKTOP_TYPE_ASH); 189 Browser::CreateParams::CreateForApp(
190 browser_create_params.app_name = "Test"; 190 Browser::TYPE_HOSTED_POPUP, "Test", gfx::Rect(),
191 browser()->profile(), chrome::HOST_DESKTOP_TYPE_ASH));
191 192
192 Browser* app_host_browser = new Browser(browser_create_params); 193 Browser* app_host_browser = new Browser(browser_create_params);
193 ASSERT_TRUE(app_host_browser->is_app()); 194 ASSERT_TRUE(app_host_browser->is_app());
194 AddBlankTabAndShow(app_host_browser); 195 AddBlankTabAndShow(app_host_browser);
195 window_state = 196 window_state =
196 ash::wm::GetWindowState(app_host_browser->window()->GetNativeWindow()); 197 ash::wm::GetWindowState(app_host_browser->window()->GetNativeWindow());
197 ASSERT_TRUE(window_state->IsActive()); 198 ASSERT_TRUE(window_state->IsActive());
198 SetToInitialShowState(window_state); 199 SetToInitialShowState(window_state);
199 EXPECT_TRUE(IsInitialShowState(window_state)); 200 EXPECT_TRUE(IsInitialShowState(window_state));
200 201
201 ash::accelerators::ToggleFullscreen(); 202 ash::accelerators::ToggleFullscreen();
202 EXPECT_TRUE(window_state->IsFullscreen()); 203 EXPECT_TRUE(window_state->IsFullscreen());
203 EXPECT_EQ(put_all_windows_in_immersive(), 204 EXPECT_EQ(put_all_windows_in_immersive(),
204 IsInImmersiveFullscreen(window_state)); 205 IsInImmersiveFullscreen(window_state));
205 206
206 ash::accelerators::ToggleFullscreen(); 207 ash::accelerators::ToggleFullscreen();
207 EXPECT_TRUE(IsInitialShowState(window_state)); 208 EXPECT_TRUE(IsInitialShowState(window_state));
208 209
209 // 4) Popup browser windows. 210 // 4) Popup browser windows.
210 browser_create_params.app_name = ""; 211 browser_create_params = Browser::CreateParams(
212 Browser::TYPE_POPUP, browser()->profile(), chrome::HOST_DESKTOP_TYPE_ASH);
211 Browser* popup_browser = new Browser(browser_create_params); 213 Browser* popup_browser = new Browser(browser_create_params);
212 ASSERT_TRUE(popup_browser->is_type_popup()); 214 ASSERT_TRUE(popup_browser->is_type_popup());
213 ASSERT_FALSE(popup_browser->is_app()); 215 ASSERT_FALSE(popup_browser->is_app());
214 AddBlankTabAndShow(popup_browser); 216 AddBlankTabAndShow(popup_browser);
215 window_state = 217 window_state =
216 ash::wm::GetWindowState(popup_browser->window()->GetNativeWindow()); 218 ash::wm::GetWindowState(popup_browser->window()->GetNativeWindow());
217 ASSERT_TRUE(window_state->IsActive()); 219 ASSERT_TRUE(window_state->IsActive());
218 SetToInitialShowState(window_state); 220 SetToInitialShowState(window_state);
219 EXPECT_TRUE(IsInitialShowState(window_state)); 221 EXPECT_TRUE(IsInitialShowState(window_state));
220 222
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
403 // cycles. 405 // cycles.
404 INSTANTIATE_TEST_CASE_P(InitiallyRestored, 406 INSTANTIATE_TEST_CASE_P(InitiallyRestored,
405 AcceleratorCommandsPlatformAppFullscreenBrowserTest, 407 AcceleratorCommandsPlatformAppFullscreenBrowserTest,
406 Combine(Values(false), 408 Combine(Values(false),
407 Values(ui::SHOW_STATE_NORMAL))); 409 Values(ui::SHOW_STATE_NORMAL)));
408 INSTANTIATE_TEST_CASE_P(InitiallyMaximized, 410 INSTANTIATE_TEST_CASE_P(InitiallyMaximized,
409 AcceleratorCommandsPlatformAppFullscreenBrowserTest, 411 AcceleratorCommandsPlatformAppFullscreenBrowserTest,
410 Combine(Values(false), 412 Combine(Values(false),
411 Values(ui::SHOW_STATE_MAXIMIZED))); 413 Values(ui::SHOW_STATE_MAXIMIZED)));
412 #endif 414 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698