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

Side by Side Diff: chrome/browser/extensions/extension_tabs_apitest.cc

Issue 184903003: Window ownership -> WindowTreeHost (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . 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 (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/extensions/extension_apitest.h" 5 #include "chrome/browser/extensions/extension_apitest.h"
6 6
7 #include "base/prefs/pref_service.h" 7 #include "base/prefs/pref_service.h"
8 #include "chrome/browser/prefs/incognito_mode_prefs.h" 8 #include "chrome/browser/prefs/incognito_mode_prefs.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/ui/browser.h" 10 #include "chrome/browser/ui/browser.h"
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 ASSERT_TRUE(RunExtensionTest("tabs/no_permissions")) << message_; 219 ASSERT_TRUE(RunExtensionTest("tabs/no_permissions")) << message_;
220 } 220 }
221 221
222 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, 222 IN_PROC_BROWSER_TEST_F(ExtensionApiTest,
223 MAYBE_UpdateWindowResize) { 223 MAYBE_UpdateWindowResize) {
224 ASSERT_TRUE(RunExtensionTest("window_update/resize")) << message_; 224 ASSERT_TRUE(RunExtensionTest("window_update/resize")) << message_;
225 } 225 }
226 226
227 #if defined(OS_WIN) 227 #if defined(OS_WIN)
228 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, FocusWindowDoesNotUnmaximize) { 228 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, FocusWindowDoesNotUnmaximize) {
229 HWND window = browser()->window()->GetNativeWindow()-> 229 HWND window =
230 GetDispatcher()->host()->GetAcceleratedWidget(); 230 browser()->window()->GetNativeWindow()->GetHost()->GetAcceleratedWidget();
231 ::SendMessage(window, WM_SYSCOMMAND, SC_MAXIMIZE, 0); 231 ::SendMessage(window, WM_SYSCOMMAND, SC_MAXIMIZE, 0);
232 ASSERT_TRUE(RunExtensionTest("window_update/focus")) << message_; 232 ASSERT_TRUE(RunExtensionTest("window_update/focus")) << message_;
233 ASSERT_TRUE(::IsZoomed(window)); 233 ASSERT_TRUE(::IsZoomed(window));
234 } 234 }
235 #endif // OS_WIN 235 #endif // OS_WIN
236 236
237 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_UpdateWindowShowState) { 237 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_UpdateWindowShowState) {
238 ASSERT_TRUE(RunExtensionTest("window_update/show_state")) << message_; 238 ASSERT_TRUE(RunExtensionTest("window_update/show_state")) << message_;
239 } 239 }
240 240
(...skipping 11 matching lines...) Expand all
252 << message_; 252 << message_;
253 } 253 }
254 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_GetViewsOfCreatedWindow) { 254 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_GetViewsOfCreatedWindow) {
255 ASSERT_TRUE(RunExtensionSubtest("tabs/basics", "get_views_window.html")) 255 ASSERT_TRUE(RunExtensionSubtest("tabs/basics", "get_views_window.html"))
256 << message_; 256 << message_;
257 } 257 }
258 258
259 // Adding a new test? Awesome. But API tests are the old hotness. The 259 // Adding a new test? Awesome. But API tests are the old hotness. The
260 // new hotness is extension_test_utils. See tabs_test.cc for an example. 260 // new hotness is extension_test_utils. See tabs_test.cc for an example.
261 // We are trying to phase out many uses of API tests as they tend to be flaky. 261 // We are trying to phase out many uses of API tests as they tend to be flaky.
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/input/input.cc ('k') | chrome/browser/first_run/try_chrome_dialog_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698