AppCacheUITests

chrome/browser/appcache/appcache_ui_test.cc

Index: chrome/browser/appcache/appcache_ui_test.cc
===================================================================
--- chrome/browser/appcache/appcache_ui_test.cc	(revision 68540)
+++ chrome/browser/appcache/appcache_ui_test.cc	(working copy)
@@ -6,16 +6,31 @@
 #include "chrome/test/ui/ui_layout_test.h"
 
 class AppCacheUITest : public UILayoutTest {
+ public:
+  void RunAppCacheTests(const char* tests[], int num_tests) {
+    FilePath http_test_dir;
+    http_test_dir = http_test_dir.AppendASCII("http");
+    http_test_dir = http_test_dir.AppendASCII("tests");
+
+    FilePath appcache_test_dir;
+    appcache_test_dir = appcache_test_dir.AppendASCII("appcache");
+    InitializeForLayoutTest(http_test_dir, appcache_test_dir, kHttpPort);
+
+    StartHttpServer(new_http_root_dir_);
+    for (int i = 0; i < num_tests; ++i)
+      RunLayoutTest(tests[i], kHttpPort);
+    StopHttpServer();
+  }
+
  protected:
   virtual ~AppCacheUITest() {}
 };
 
-// Flaky: http://crbug.com/54717
-TEST_F(AppCacheUITest, FLAKY_AppCacheLayoutTests) {
-  static const char* kLayoutTestFiles[] = {
+// The tests that don't depend on PHP should be less flaky.

[Paweł Hajdan Jr., 2010/12/10 18:37:05]

nit: If you keep the FLAKY prefix, please also keep the bug link. Otherwise,
remove the FLAKY prefix (up to you).

+TEST_F(AppCacheUITest, FLAKY_AppCacheLayoutTests_NoPHP) {
+  static const char* kNoPHPTests[] = {
       "404-manifest.html",
       "404-resource.html",
-      "auth.html",
       "cyrillic-uri.html",
       "deferred-events-delete-while-raising.html",
       "deferred-events.html",
@@ -24,22 +39,14 @@
       "different-origin-manifest.html",
       "different-scheme.html",
       "empty-manifest.html",
-      "fallback.html",
       "foreign-iframe-main.html",
-      "main-resource-hash.html",
+      "insert-html-element-with-manifest.html",
+      "insert-html-element-with-manifest-2.html",
       "manifest-containing-itself.html",
       "manifest-parsing.html",
-      "manifest-redirect-2.html",
-      "manifest-redirect.html",
       "manifest-with-empty-file.html",
-      "navigating-away-while-cache-attempt-in-progress.html",
-      "offline-access.html",
-      "online-whitelist.html",
       "progress-counter.html",
       "reload.html",
-      "remove-cache.html",
-      "resource-redirect-2.html",
-      "resource-redirect.html",
       "simple.html",
       "top-frame-1.html",
       "top-frame-2.html",
@@ -50,28 +57,41 @@
       "wrong-signature-2.html",
       "wrong-signature.html",
       "xhr-foreign-resource.html",
+  };
+  // "foreign-fallback.html",  // This test is racey.
 
-      // TODO(michaeln): investigate these more closely
-      // "crash-when-navigating-away-then-back.html",
-      // "credential-url.html",
-      // "different-https-origin-resource-main.html",
-      // "fail-on-update.html",
-      // "idempotent-update.html", not sure this is a valid test
-      // "local-content.html",
-      // "max-size.html", we use a different quota scheme
-      // "update-cache.html",  bug 38006
+  RunAppCacheTests(kNoPHPTests, arraysize(kNoPHPTests));
+}
+
+// Flaky: http://crbug.com/54717
+// Lighty/PHP is not reliable enough on windows.
+TEST_F(AppCacheUITest, FLAKY_AppCacheLayoutTests_PHP) {
+  static const char* kPHPTests[] = {
+      "auth.html",
+      "fallback.html",
+      "main-resource-hash.html",
+      "manifest-redirect.html",
+      "manifest-redirect-2.html",
+      "navigating-away-while-cache-attempt-in-progress.html",
+      "non-html.xhtml",
+      "offline-access.html",
+      "online-whitelist.html",
+      "resource-redirect.html",
+      "resource-redirect-2.html",
+      "update-cache.html",
   };
 
-  FilePath http_test_dir;
-  http_test_dir = http_test_dir.AppendASCII("http");
-  http_test_dir = http_test_dir.AppendASCII("tests");
+  // "fail-on-update.html",  // This test is racey, status polling.
+  // "fail-on-update2.html",  // This test is racey, status polling.
+  // "remove-cache.html",  // This test is racey, status polling.
 
-  FilePath appcache_test_dir;
-  appcache_test_dir = appcache_test_dir.AppendASCII("appcache");
-  InitializeForLayoutTest(http_test_dir, appcache_test_dir, kHttpPort);
+  // TODO(michaeln): investigate these more closely
+  // "crash-when-navigating-away-then-back.html",
+  // "credential-url.html",
+  // "different-https-origin-resource-main.html",
+  // "idempotent-update.html", not sure this is a valid test
+  // "local-content.html",
+  // "max-size.html", we use a different quota scheme
 
-  StartHttpServer(new_http_root_dir_);
-  for (size_t i = 0; i < arraysize(kLayoutTestFiles); ++i)
-    RunLayoutTest(kLayoutTestFiles[i], kHttpPort);
-  StopHttpServer();
+  RunAppCacheTests(kPHPTests, arraysize(kPHPTests));
 }