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

Side by Side Diff: chrome/browser/policy/policy_browsertest.cc

Issue 2248873002: Convert WindowOpenDisposition to an enum class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 3 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
OLDNEW
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 <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include <algorithm> 8 #include <algorithm>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 2527 matching lines...) Expand 10 before | Expand all | Expand 10 after
2538 base::WrapUnique(new base::FundamentalValue(false)), nullptr); 2538 base::WrapUnique(new base::FundamentalValue(false)), nullptr);
2539 UpdateProviderPolicy(policies); 2539 UpdateProviderPolicy(policies);
2540 2540
2541 const extensions::Extension* extension = 2541 const extensions::Extension* extension =
2542 LoadUnpackedExtension(kUnpackedFullscreenAppName, true); 2542 LoadUnpackedExtension(kUnpackedFullscreenAppName, true);
2543 ASSERT_TRUE(extension); 2543 ASSERT_TRUE(extension);
2544 2544
2545 // Launch an app that tries to open a fullscreen window. 2545 // Launch an app that tries to open a fullscreen window.
2546 TestAddAppWindowObserver add_window_observer( 2546 TestAddAppWindowObserver add_window_observer(
2547 extensions::AppWindowRegistry::Get(browser()->profile())); 2547 extensions::AppWindowRegistry::Get(browser()->profile()));
2548 OpenApplication(AppLaunchParams(browser()->profile(), extension, 2548 OpenApplication(AppLaunchParams(
2549 extensions::LAUNCH_CONTAINER_NONE, NEW_WINDOW, 2549 browser()->profile(), extension, extensions::LAUNCH_CONTAINER_NONE,
2550 extensions::SOURCE_TEST)); 2550 WindowOpenDisposition::NEW_WINDOW, extensions::SOURCE_TEST));
2551 extensions::AppWindow* window = add_window_observer.WaitForAppWindow(); 2551 extensions::AppWindow* window = add_window_observer.WaitForAppWindow();
2552 ASSERT_TRUE(window); 2552 ASSERT_TRUE(window);
2553 2553
2554 // Verify that the window is not in fullscreen mode. 2554 // Verify that the window is not in fullscreen mode.
2555 EXPECT_FALSE(window->GetBaseWindow()->IsFullscreen()); 2555 EXPECT_FALSE(window->GetBaseWindow()->IsFullscreen());
2556 2556
2557 // Verify that the window cannot be toggled into fullscreen mode via apps 2557 // Verify that the window cannot be toggled into fullscreen mode via apps
2558 // APIs. 2558 // APIs.
2559 EXPECT_TRUE(content::ExecuteScript( 2559 EXPECT_TRUE(content::ExecuteScript(
2560 window->web_contents(), 2560 window->web_contents(),
(...skipping 1825 matching lines...) Expand 10 before | Expand all | Expand 10 after
4386 4386
4387 SetEmptyPolicy(); 4387 SetEmptyPolicy();
4388 // Policy not set. 4388 // Policy not set.
4389 CheckSystemTimezoneAutomaticDetectionPolicyUnset(); 4389 CheckSystemTimezoneAutomaticDetectionPolicyUnset();
4390 EXPECT_TRUE(CheckResolveTimezoneByGeolocation(true, false)); 4390 EXPECT_TRUE(CheckResolveTimezoneByGeolocation(true, false));
4391 EXPECT_TRUE(manager->TimeZoneResolverShouldBeRunningForTests()); 4391 EXPECT_TRUE(manager->TimeZoneResolverShouldBeRunningForTests());
4392 } 4392 }
4393 #endif // defined(OS_CHROMEOS) 4393 #endif // defined(OS_CHROMEOS)
4394 4394
4395 } // namespace policy 4395 } // namespace policy
OLDNEW
« no previous file with comments | « chrome/browser/plugins/plugin_power_saver_browsertest.cc ('k') | chrome/browser/prerender/prerender_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698