Update AbstractWorker, Worker and SharedWorker to match the spec

Source/core/workers/AbstractWorker.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 14 matching lines @@
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef AbstractWorker_h
 #define AbstractWorker_h
 
 #include "bindings/v8/ScriptWrappable.h"
-#include "core/dom/ActiveDOMObject.h"
 #include "core/dom/EventListener.h"
 #include "core/dom/EventNames.h"
 #include "core/dom/EventTarget.h"
 #include <wtf/PassRefPtr.h>
 #include <wtf/RefCounted.h>
 #include <wtf/RefPtr.h>
 #include <wtf/text/AtomicStringHash.h>
 
 namespace WebCore {
 
     class KURL;
     class ScriptExecutionContext;
 
-    class AbstractWorker : public RefCounted<AbstractWorker>, public ScriptWrappable, public ActiveDOMObject, public EventTarget {

[haraken, 2013/06/19 10:24:00]

Why did you move ScriptWrappable and ActiveDOMObject from AbstractWorker to
Worker and SharedWorker? It will be harmless but I'm curious about the
background.

[do-not-use, 2013/06/19 10:39:09]

It did not link if AbstractWorker would subclass ScriptWrappable in C++ because
of undefined
webCoreInitializeScriptWrappableForInterface(WebCore::AbstractWorker*). This is
because there is no longer any code generated for AbstractWorker.idl (No
JSAbstractWorker).

Regarding ActiveDOMObject, I was hitting assertions in the RefPtr unref without
the change (deletionHasBegun). I did not investigate yet why this happens but
since [ActiveDOMObject] was removed from AbstractWorker.idl, I figured this was
linked and it did fix the crashes.

[do-not-use, 2013/06/19 10:52:08]

I'll try to move back ActiveDOMObject to AbstractWorker.h to make sure the move
is really needed. 

+    class AbstractWorker : public RefCounted<AbstractWorker>, public EventTarget {
     public:
         // EventTarget APIs
-        virtual ScriptExecutionContext* scriptExecutionContext() const OVERRIDE { return ActiveDOMObject::scriptExecutionContext(); }
 
         DEFINE_ATTRIBUTE_EVENT_LISTENER(error);
 
         using RefCounted<AbstractWorker>::ref;
         using RefCounted<AbstractWorker>::deref;
 
-        virtual void contextDestroyed() OVERRIDE;
-        AbstractWorker(ScriptExecutionContext*);
+        AbstractWorker();
         virtual ~AbstractWorker();
 
     protected:
         // Helper function that converts a URL to an absolute URL and checks the result for validity.
         KURL resolveURL(const String& url, ExceptionCode& ec);
         intptr_t asID() const { return reinterpret_cast<intptr_t>(this); }
 
     private:
         virtual void refEventTarget() OVERRIDE { ref(); }
         virtual void derefEventTarget() OVERRIDE { deref(); }
         virtual EventTargetData* eventTargetData() OVERRIDE;
         virtual EventTargetData* ensureEventTargetData() OVERRIDE;
         
         EventTargetData m_eventTargetData;
     };
 
 } // namespace WebCore
 
 #endif // AbstractWorker_h