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

Side by Side Diff: chrome/test/plugin/plugin_test.cpp

Issue 4135007: Add a bunch of missing bug references to FLAKY tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add TODO Created 10 years, 1 month 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
« no previous file with comments | « chrome/browser/unload_uitest.cc ('k') | chrome/test/ui/layout_plugin_uitest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 // Tests for the top plugins to catch regressions in our plugin host code, as 5 // Tests for the top plugins to catch regressions in our plugin host code, as
6 // well as in the out of process code. Currently this tests: 6 // well as in the out of process code. Currently this tests:
7 // Flash 7 // Flash
8 // Real 8 // Real
9 // QuickTime 9 // QuickTime
10 // Windows Media Player 10 // Windows Media Player
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 #if defined(OS_WIN) 186 #if defined(OS_WIN)
187 // Windows only test 187 // Windows only test
188 TEST_F(PluginTest, FlashSecurity) { 188 TEST_F(PluginTest, FlashSecurity) {
189 TestPlugin("flash.html", action_max_timeout_ms(), false); 189 TestPlugin("flash.html", action_max_timeout_ms(), false);
190 } 190 }
191 #endif // defined(OS_WIN) 191 #endif // defined(OS_WIN)
192 192
193 #if defined(OS_WIN) 193 #if defined(OS_WIN)
194 // TODO(port) Port the following tests to platforms that have the required 194 // TODO(port) Port the following tests to platforms that have the required
195 // plugins. 195 // plugins.
196 // Flaky: http://crbug.com/55915
196 TEST_F(PluginTest, FLAKY_Quicktime) { 197 TEST_F(PluginTest, FLAKY_Quicktime) {
197 TestPlugin("quicktime.html", action_max_timeout_ms(), false); 198 TestPlugin("quicktime.html", action_max_timeout_ms(), false);
198 } 199 }
199 200
200 // Disabled on Release bots - http://crbug.com/44662 201 // Disabled on Release bots - http://crbug.com/44662
201 #if defined(NDEBUG) 202 #if defined(NDEBUG)
202 #define MediaPlayerNew DISABLED_MediaPlayerNew 203 #define MediaPlayerNew DISABLED_MediaPlayerNew
203 #endif 204 #endif
204 TEST_F(PluginTest, MediaPlayerNew) { 205 TEST_F(PluginTest, MediaPlayerNew) {
205 TestPlugin("wmp_new.html", action_max_timeout_ms(), false); 206 TestPlugin("wmp_new.html", action_max_timeout_ms(), false);
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
343 TEST_F(PluginInstallerDownloadTest, PluginInstallerDownloadPathTest) { 344 TEST_F(PluginInstallerDownloadTest, PluginInstallerDownloadPathTest) {
344 MessageLoop loop(MessageLoop::TYPE_IO); 345 MessageLoop loop(MessageLoop::TYPE_IO);
345 Start(); 346 Start();
346 loop.Run(); 347 loop.Run();
347 348
348 EXPECT_TRUE(success()); 349 EXPECT_TRUE(success());
349 EXPECT_TRUE(initial_download_path().BaseName().value() == 350 EXPECT_TRUE(initial_download_path().BaseName().value() ==
350 final_download_path().BaseName().value()); 351 final_download_path().BaseName().value());
351 } 352 }
352 #endif // defined(OS_WIN) 353 #endif // defined(OS_WIN)
OLDNEW
« no previous file with comments | « chrome/browser/unload_uitest.cc ('k') | chrome/test/ui/layout_plugin_uitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698