Disable all the tests that are flaking more than 5% on Windows builders.

content/browser/plugin_browsertest.cc

Index: content/browser/plugin_browsertest.cc
diff --git a/content/browser/plugin_browsertest.cc b/content/browser/plugin_browsertest.cc
index 223b7245fdaa4bf1949f1ed5dcf969e19654f1c4..d2c66fd0a5ee3e8ebfca8c8d5589eac20c2d44e4 100644
--- a/content/browser/plugin_browsertest.cc
+++ b/content/browser/plugin_browsertest.cc
@@ -354,6 +354,7 @@ IN_PROC_BROWSER_TEST_F(PluginTest, PluginSingleRangeRequest) {
   LoadAndWait(GetURL("plugin_single_range_request.html"));
 }
 
+#if !defined(OS_WIN) // http://crbug.com/396373

[viettrungluu, 2014/07/23 16:18:52]

Probably you should just define this as MAYBE_PrivateEnabled and define that to
be DISABLED_... on Windows (and get rid of the use of Justin's confusingly-named
MAYBE macro).

[jam, 2014/07/23 16:30:10]

the complexity is because of that x64 macro. if i make it MAYBE_, then if
someone fixes it on windows they might not use the MAYBE() and then it'll fail
on x64. so that's why i disabled it this way.

[viettrungluu, 2014/07/23 16:38:50]

Right, but the problem with just ifdef-ing them out is that they won't even be
compiled anymore (so compilation may be broken) and they won't show up in lists
of disabled tests.

I figure than turning them to standard MAYBE_.../DISABLED_... is okay, because
anyone who tries to reenable them would then immediately notice that it's broken
on Win x64. (Or maybe stuff will have been implemented on x64 by then.)

 // Test checking the privacy mode is on.
 // If this flakes on Linux, use http://crbug.com/104380
 IN_PROC_BROWSER_TEST_F(PluginTest, MAYBE(PrivateEnabled)) {
@@ -361,14 +362,15 @@ IN_PROC_BROWSER_TEST_F(PluginTest, MAYBE(PrivateEnabled)) {
   url = GURL(url.spec() + "?private");
   LoadAndWaitInWindow(CreateOffTheRecordBrowser(), url);
 }
+#endif
 
-#if defined(OS_WIN) || defined(OS_MACOSX)
+// These used to run on Windows: http://crbug.com/396373

[viettrungluu, 2014/07/23 16:18:52]

"

+#if defined(OS_MACOSX)
 // Test a browser hang due to special case of multiple
 // plugin instances indulged in sync calls across renderer.
 IN_PROC_BROWSER_TEST_F(PluginTest, MAYBE(MultipleInstancesSyncCalls)) {
   LoadAndWait(GetURL("multiple_instances_sync_calls.html"));
 }
-#endif
 
 IN_PROC_BROWSER_TEST_F(PluginTest, MAYBE(GetURLRequestFailWrite)) {
   GURL url(URLRequestMockHTTPJob::GetMockUrl(
@@ -376,6 +378,7 @@ IN_PROC_BROWSER_TEST_F(PluginTest, MAYBE(GetURLRequestFailWrite)) {
                        AppendASCII("plugin_url_request_fail_write.html")));
   LoadAndWait(url);
 }
+#endif
 
 #if defined(OS_WIN)
 // Flaky on Windows x86.  http://crbug.com/388245