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

Unified Diff: chrome/worker/worker_uitest.cc

Issue 390017: Added lifecycle management and sharing support for SharedWorkers. SharedWorkers (Closed)
Patch Set: Changed WebWorkerBase not not call a virtual function from the destructor Created 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/worker/worker_uitest.cc
diff --git a/chrome/worker/worker_uitest.cc b/chrome/worker/worker_uitest.cc
index 793f8893f900ed416ed1479ccd620bb4677cfdf1..66c8ce42c30ecfc8fb9a627862e13a458184e0ab 100644
--- a/chrome/worker/worker_uitest.cc
+++ b/chrome/worker/worker_uitest.cc
@@ -119,22 +119,22 @@ TEST_F(WorkerTest, WorkerFastLayoutTests) {
TEST_F(WorkerTest, SharedWorkerFastLayoutTests) {
static const char* kLayoutTestFiles[] = {
"shared-worker-constructor.html",
- // Enable remaining SharedWorker tests when functionality is
- // complete (http://crbug.com/26899)
"shared-worker-context-gc.html",
"shared-worker-event-listener.html",
- //"shared-worker-exception.html",
- //"shared-worker-frame-lifecycle.html",
+ "shared-worker-exception.html",
"shared-worker-gc.html",
+ // Lifecycle tests rely on layoutTestController.workerThreadCount which is
+ // not currently implemented.
+ //"shared-worker-frame-lifecycle.html",
//"shared-worker-lifecycle.html",
"shared-worker-load-error.html",
"shared-worker-location.html",
- //"shared-worker-name.html",
+ "shared-worker-name.html",
"shared-worker-navigator.html",
"shared-worker-replace-global-constructor.html",
"shared-worker-replace-self.html",
"shared-worker-script-error.html",
- //"shared-worker-shared.html",
+ "shared-worker-shared.html",
"shared-worker-simple.html",
};

Powered by Google App Engine
This is Rietveld 408576698