compositor-worker: Restrict the global interface in CompositorWorker.

LayoutTests/resources/js-test.js

Index: LayoutTests/resources/js-test.js
diff --git a/LayoutTests/resources/js-test.js b/LayoutTests/resources/js-test.js
index e86533fc2bd1befb5085f7abbadbcc0118352dc5..156ba5ec1e4b75fbf81bef658a944925ef95cdd6 100644
--- a/LayoutTests/resources/js-test.js
+++ b/LayoutTests/resources/js-test.js
@@ -811,11 +811,13 @@ function finishJSTest()
         testRunner.notifyDone();
 }
 
-function startWorker(testScriptURL, shared)
+function startWorker(testScriptURL, workerType)
 {
+    var shared = (workerType == 'shared' || (typeof workerType == 'boolean' && workerType));

[haraken, 2015/04/22 11:07:28]

What is the 'typeof workerType == 'boolean' && workerType' check for?

[kinuko, 2015/04/22 14:18:07]

Can we just update all startWorker() for shared worker to take string? (I
guess/hope there're not too many tests for shared worker)

[sadrul, 2015/04/22 22:17:19]

Done! (there was certainly way less than I feared)

+    var compositor = workerType == 'compositor';
     self.jsTestIsAsync = true;
     debug('Starting worker: ' + testScriptURL);
-    var worker = shared ? new SharedWorker(testScriptURL, "Shared Worker") : new Worker(testScriptURL);
+    var worker = shared ? new SharedWorker(testScriptURL, "Shared Worker") : compositor ? new CompositorWorker(testScriptURL) : new Worker(testScriptURL);

[haraken, 2015/04/22 11:07:28]

Can we clean up the code a bit more?

if (...)
  worker = new CompositorWorker();
else if (...)
  worker = new SharedWorker();
else
  worker = new Worker();

[sadrul, 2015/04/22 22:17:19]

Done.

     worker.onmessage = function(event)
     {
         var workerPrefix = "[Worker] ";