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

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

Issue 11091072: Fix failing browser tests when running with force-compositing-mode. It uses new trace events checke… (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 2 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 "chrome/browser/extensions/extension_apitest.h" 5 #include "chrome/browser/extensions/extension_apitest.h"
6 6
7 #include "chrome/browser/prefs/incognito_mode_prefs.h" 7 #include "chrome/browser/prefs/incognito_mode_prefs.h"
8 #include "chrome/browser/prefs/pref_service.h" 8 #include "chrome/browser/prefs/pref_service.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 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 } 136 }
137 137
138 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_TabOnRemoved) { 138 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_TabOnRemoved) {
139 ASSERT_TRUE(RunExtensionTest("tabs/on_removed")) << message_; 139 ASSERT_TRUE(RunExtensionTest("tabs/on_removed")) << message_;
140 } 140 }
141 141
142 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_TabReload) { 142 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_TabReload) {
143 ASSERT_TRUE(RunExtensionTest("tabs/reload")) << message_; 143 ASSERT_TRUE(RunExtensionTest("tabs/reload")) << message_;
144 } 144 }
145 145
146 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, CaptureVisibleTabJpeg) { 146 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_CaptureVisibleTabJpeg) {
147 host_resolver()->AddRule("a.com", "127.0.0.1"); 147 host_resolver()->AddRule("a.com", "127.0.0.1");
148 host_resolver()->AddRule("b.com", "127.0.0.1"); 148 host_resolver()->AddRule("b.com", "127.0.0.1");
149 ASSERT_TRUE(StartTestServer()); 149 ASSERT_TRUE(StartTestServer());
150 ASSERT_TRUE(RunExtensionSubtest("tabs/capture_visible_tab", 150 ASSERT_TRUE(RunExtensionSubtest("tabs/capture_visible_tab",
151 "test_jpeg.html")) << message_; 151 "test_jpeg.html")) << message_;
152 } 152 }
153 153
154 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_CaptureVisibleTabPng) { 154 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_CaptureVisibleTabPng) {
155 host_resolver()->AddRule("a.com", "127.0.0.1"); 155 host_resolver()->AddRule("a.com", "127.0.0.1");
156 host_resolver()->AddRule("b.com", "127.0.0.1"); 156 host_resolver()->AddRule("b.com", "127.0.0.1");
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 << message_; 237 << message_;
238 } 238 }
239 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_GetViewsOfCreatedWindow) { 239 IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_GetViewsOfCreatedWindow) {
240 ASSERT_TRUE(RunExtensionSubtest("tabs/basics", "get_views_window.html")) 240 ASSERT_TRUE(RunExtensionSubtest("tabs/basics", "get_views_window.html"))
241 << message_; 241 << message_;
242 } 242 }
243 243
244 // Adding a new test? Awesome. But API tests are the old hotness. The 244 // Adding a new test? Awesome. But API tests are the old hotness. The
245 // new hotness is extension_test_utils. See tabs_test.cc for an example. 245 // new hotness is extension_test_utils. See tabs_test.cc for an example.
246 // We are trying to phase out many uses of API tests as they tend to be flaky. 246 // 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/chrome_gpu_util.cc ('k') | chrome/browser/task_manager/task_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698