Minor cleanup in worker code
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Mar 2013 19:03:39 +0000 (19:03 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Mar 2013 19:03:39 +0000 (19:03 +0000)
        https://bugs.webkit.org/show_bug.cgi?id=112455

        Reviewed by Anders Carlsson.

        - Added OVERRIDE keywords.
        - Toned down FIXMEs about single MessagePort versions of functions. It's not a given
        that we'll want to get rid of them even when implementing array versions.
        - Explicitly marked virtual destructors as virtual.

        * dom/MessageEvent.cpp:
        * dom/MessageEvent.h:
        * workers/Worker.cpp:
        * workers/Worker.h:
        Removed an unused version of postMessage() that took no MessagePorts. This makes
        it less confusing why MessagePortArray is passed by pointer, and not by reference.

        * workers/DedicatedWorkerContext.cpp:
        (WebCore::DedicatedWorkerContext::~DedicatedWorkerContext):
        * workers/DedicatedWorkerContext.h:
        Added a destructor instead of an implicit one.
        Removed an unused version of postMessage() that took no MessagePorts.

        * workers/SharedWorker.h: Removed an unused virtual function (thank you OVERRIDE!)

        * dom/MessagePort.cpp:
        * dom/MessagePort.h:
        * loader/WorkerThreadableLoader.h:
        * page/DOMWindow.h:
        * workers/AbstractWorker.h:
        * workers/DedicatedWorkerThread.h:
        * workers/SharedWorkerContext.h:
        * workers/SharedWorkerThread.h:
        * workers/WorkerContext.h:
        * workers/WorkerMessagingProxy.h:
        * workers/WorkerObjectProxy.h:
        * workers/WorkerScriptLoader.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@145930 268f45cc-cd09-0410-ab3c-d52691b4dbfc

20 files changed:
Source/WebCore/ChangeLog
Source/WebCore/dom/MessageEvent.cpp
Source/WebCore/dom/MessageEvent.h
Source/WebCore/dom/MessagePort.cpp
Source/WebCore/dom/MessagePort.h
Source/WebCore/loader/WorkerThreadableLoader.h
Source/WebCore/page/DOMWindow.h
Source/WebCore/workers/AbstractWorker.h
Source/WebCore/workers/DedicatedWorkerContext.cpp
Source/WebCore/workers/DedicatedWorkerContext.h
Source/WebCore/workers/DedicatedWorkerThread.h
Source/WebCore/workers/SharedWorker.h
Source/WebCore/workers/SharedWorkerContext.h
Source/WebCore/workers/SharedWorkerThread.h
Source/WebCore/workers/Worker.cpp
Source/WebCore/workers/Worker.h
Source/WebCore/workers/WorkerContext.h
Source/WebCore/workers/WorkerMessagingProxy.h
Source/WebCore/workers/WorkerObjectProxy.h
Source/WebCore/workers/WorkerScriptLoader.h

index ab410e6..5a06ed3 100644 (file)
@@ -1,3 +1,43 @@
+2013-03-15  Alexey Proskuryakov  <ap@apple.com>
+
+        Minor cleanup in worker code
+        https://bugs.webkit.org/show_bug.cgi?id=112455
+
+        Reviewed by Anders Carlsson.
+
+        - Added OVERRIDE keywords.
+        - Toned down FIXMEs about single MessagePort versions of functions. It's not a given
+        that we'll want to get rid of them even when implementing array versions.
+        - Explicitly marked virtual destructors as virtual.
+
+        * dom/MessageEvent.cpp:
+        * dom/MessageEvent.h:
+        * workers/Worker.cpp:
+        * workers/Worker.h:
+        Removed an unused version of postMessage() that took no MessagePorts. This makes
+        it less confusing why MessagePortArray is passed by pointer, and not by reference.
+
+        * workers/DedicatedWorkerContext.cpp:
+        (WebCore::DedicatedWorkerContext::~DedicatedWorkerContext):
+        * workers/DedicatedWorkerContext.h:
+        Added a destructor instead of an implicit one.
+        Removed an unused version of postMessage() that took no MessagePorts.
+
+        * workers/SharedWorker.h: Removed an unused virtual function (thank you OVERRIDE!)
+
+        * dom/MessagePort.cpp:
+        * dom/MessagePort.h:
+        * loader/WorkerThreadableLoader.h:
+        * page/DOMWindow.h:
+        * workers/AbstractWorker.h:
+        * workers/DedicatedWorkerThread.h:
+        * workers/SharedWorkerContext.h:
+        * workers/SharedWorkerThread.h:
+        * workers/WorkerContext.h:
+        * workers/WorkerMessagingProxy.h:
+        * workers/WorkerObjectProxy.h:
+        * workers/WorkerScriptLoader.h:
+
 2013-03-15  Joshua Bell  <jsbell@chromium.org>
 
         [V8] Binding: Implement EnforceRange IDL Attribute for long long conversions
index 1282a25..6cdaac1 100644 (file)
@@ -153,7 +153,6 @@ SerializedScriptValue* MessageEvent::data() const
     return m_dataAsSerializedScriptValue.get();
 }
 
-// FIXME: remove this when we update the ObjC bindings (bug #28774).
 MessagePort* MessageEvent::messagePort()
 {
     if (!m_ports)
@@ -162,7 +161,6 @@ MessagePort* MessageEvent::messagePort()
     return (*m_ports)[0].get();
 }
 
-// FIXME: remove this when we update the ObjC bindings (bug #28774).
 void MessageEvent::initMessageEvent(const AtomicString& type, bool canBubble, bool cancelable, PassRefPtr<SerializedScriptValue> data, const String& origin, const String& lastEventId, DOMWindow* source, MessagePort* port)
 {
     OwnPtr<MessagePortArray> ports;
index 8b229b9..05e368a 100644 (file)
@@ -92,9 +92,9 @@ public:
 
     // FIXME: Remove this when we have custom ObjC binding support.
     SerializedScriptValue* data() const;
-    // FIXME: remove this when we update the ObjC bindings (bug #28774).
+
+    // Needed for Objective-C bindings (see bug 28774).
     MessagePort* messagePort();
-    // FIXME: remove this when we update the ObjC bindings (bug #28774).
     void initMessageEvent(const AtomicString& type, bool canBubble, bool cancelable, PassRefPtr<SerializedScriptValue> data, const String& origin, const String& lastEventId, DOMWindow* source, MessagePort*);
 
     virtual const AtomicString& interfaceName() const;
index 7cb72ae..0e3b923 100644 (file)
@@ -57,7 +57,6 @@ MessagePort::~MessagePort()
         m_scriptExecutionContext->destroyedMessagePort(this);
 }
 
-// FIXME: remove this when we update the ObjC bindings (bug #28774).
 void MessagePort::postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort* port, ExceptionCode& ec)
 {
     MessagePortArray ports;
@@ -66,11 +65,6 @@ void MessagePort::postMessage(PassRefPtr<SerializedScriptValue> message, Message
     postMessage(message, &ports, ec);
 }
 
-void MessagePort::postMessage(PassRefPtr<SerializedScriptValue> message, ExceptionCode& ec)
-{
-    postMessage(message, static_cast<MessagePortArray*>(0), ec);
-}
-
 void MessagePort::postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray* ports, ExceptionCode& ec)
 {
     if (!isEntangled())
index 882bd3a..8adb61e 100644 (file)
@@ -55,11 +55,10 @@ namespace WebCore {
     class MessagePort : public RefCounted<MessagePort>, public EventTarget {
     public:
         static PassRefPtr<MessagePort> create(ScriptExecutionContext& scriptExecutionContext) { return adoptRef(new MessagePort(scriptExecutionContext)); }
-        ~MessagePort();
+        virtual ~MessagePort();
 
-        void postMessage(PassRefPtr<SerializedScriptValue> message, ExceptionCode&);
         void postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray*, ExceptionCode&);
-        // FIXME: remove this when we update the ObjC bindings (bug #28774).
+        // Needed for Objective-C bindings (see bug 28774).
         void postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort*, ExceptionCode&);
 
         void start();
@@ -82,8 +81,8 @@ namespace WebCore {
 
         void contextDestroyed();
 
-        virtual const AtomicString& interfaceName() const;
-        virtual ScriptExecutionContext* scriptExecutionContext() const;
+        virtual const AtomicString& interfaceName() const OVERRIDE;
+        virtual ScriptExecutionContext* scriptExecutionContext() const OVERRIDE;
 
         void dispatchMessages();
 
@@ -111,10 +110,10 @@ namespace WebCore {
     private:
         explicit MessagePort(ScriptExecutionContext&);
 
-        virtual void refEventTarget() { ref(); }
-        virtual void derefEventTarget() { deref(); }
-        virtual EventTargetData* eventTargetData();
-        virtual EventTargetData* ensureEventTargetData();
+        virtual void refEventTarget() OVERRIDE { ref(); }
+        virtual void derefEventTarget() OVERRIDE { deref(); }
+        virtual EventTargetData* eventTargetData() OVERRIDE;
+        virtual EventTargetData* ensureEventTargetData() OVERRIDE;
 
         OwnPtr<MessagePortChannel> m_entangledChannel;
 
index aa53196..43c8358 100644 (file)
@@ -64,7 +64,7 @@ namespace WebCore {
 
         ~WorkerThreadableLoader();
 
-        virtual void cancel();
+        virtual void cancel() OVERRIDE;
 
         bool done() const { return m_workerClientWrapper->done(); }
 
@@ -72,8 +72,8 @@ namespace WebCore {
         using RefCounted<WorkerThreadableLoader>::deref;
 
     protected:
-        virtual void refThreadableLoader() { ref(); }
-        virtual void derefThreadableLoader() { deref(); }
+        virtual void refThreadableLoader() OVERRIDE { ref(); }
+        virtual void derefThreadableLoader() OVERRIDE { deref(); }
 
     private:
         // Creates a loader on the main thread and bridges communication between
@@ -112,14 +112,14 @@ namespace WebCore {
 
             static void mainThreadCreateLoader(ScriptExecutionContext*, MainThreadBridge*, PassOwnPtr<CrossThreadResourceRequestData>, ThreadableLoaderOptions, const String& outgoingReferrer);
             static void mainThreadCancel(ScriptExecutionContext*, MainThreadBridge*);
-            virtual void didSendData(unsigned long long bytesSent, unsigned long long totalBytesToBeSent);
-            virtual void didReceiveResponse(unsigned long identifier, const ResourceResponse&);
-            virtual void didReceiveData(const char*, int dataLength);
-            virtual void didReceiveCachedMetadata(const char*, int dataLength);
-            virtual void didFinishLoading(unsigned long identifier, double finishTime);
-            virtual void didFail(const ResourceError&);
-            virtual void didFailAccessControlCheck(const ResourceError&);
-            virtual void didFailRedirectCheck();
+            virtual void didSendData(unsigned long long bytesSent, unsigned long long totalBytesToBeSent) OVERRIDE;
+            virtual void didReceiveResponse(unsigned long identifier, const ResourceResponse&) OVERRIDE;
+            virtual void didReceiveData(const char*, int dataLength) OVERRIDE;
+            virtual void didReceiveCachedMetadata(const char*, int dataLength) OVERRIDE;
+            virtual void didFinishLoading(unsigned long identifier, double finishTime) OVERRIDE;
+            virtual void didFail(const ResourceError&) OVERRIDE;
+            virtual void didFailAccessControlCheck(const ResourceError&) OVERRIDE;
+            virtual void didFailRedirectCheck() OVERRIDE;
 
             // Only to be used on the main thread.
             RefPtr<ThreadableLoader> m_mainThreadLoader;
index 4b76438..49bf8c9 100644 (file)
@@ -238,7 +238,7 @@ namespace WebCore {
         String crossDomainAccessErrorMessage(DOMWindow* activeWindow);
 
         void postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray*, const String& targetOrigin, DOMWindow* source, ExceptionCode&);
-        // FIXME: remove this when we update the ObjC bindings (bug #28774).
+        // Needed for Objective-C bindings (see bug 28774).
         void postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort*, const String& targetOrigin, DOMWindow* source, ExceptionCode&);
         void postMessageTimerFired(PassOwnPtr<PostMessageTimer>);
         void dispatchMessageEventWithOriginCheck(SecurityOrigin* intendedTargetOrigin, PassRefPtr<Event>, PassRefPtr<ScriptCallStack>);
index 49754f2..7d2d930 100644 (file)
@@ -50,14 +50,14 @@ namespace WebCore {
     class AbstractWorker : public RefCounted<AbstractWorker>, public ActiveDOMObject, public EventTarget {
     public:
         // EventTarget APIs
-        virtual ScriptExecutionContext* scriptExecutionContext() const { return ActiveDOMObject::scriptExecutionContext(); }
+        virtual ScriptExecutionContext* scriptExecutionContext() const OVERRIDE { return ActiveDOMObject::scriptExecutionContext(); }
 
         DEFINE_ATTRIBUTE_EVENT_LISTENER(error);
 
         using RefCounted<AbstractWorker>::ref;
         using RefCounted<AbstractWorker>::deref;
 
-        virtual void contextDestroyed();
+        virtual void contextDestroyed() OVERRIDE;
         AbstractWorker(ScriptExecutionContext*);
         virtual ~AbstractWorker();
 
@@ -67,10 +67,10 @@ namespace WebCore {
         intptr_t asID() const { return reinterpret_cast<intptr_t>(this); }
 
     private:
-        virtual void refEventTarget() { ref(); }
-        virtual void derefEventTarget() { deref(); }
-        virtual EventTargetData* eventTargetData();
-        virtual EventTargetData* ensureEventTargetData();
+        virtual void refEventTarget() OVERRIDE { ref(); }
+        virtual void derefEventTarget() OVERRIDE { deref(); }
+        virtual EventTargetData* eventTargetData() OVERRIDE;
+        virtual EventTargetData* ensureEventTargetData() OVERRIDE;
         
         EventTargetData m_eventTargetData;
     };
index 769de8c..225915b 100644 (file)
@@ -41,7 +41,6 @@
 
 namespace WebCore {
 
-// static
 PassRefPtr<DedicatedWorkerContext> DedicatedWorkerContext::create(const KURL& url, const String& userAgent, PassOwnPtr<GroupSettings> settings, DedicatedWorkerThread* thread, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType contentSecurityPolicyType, PassRefPtr<SecurityOrigin> topOrigin)
 {
     RefPtr<DedicatedWorkerContext> context = adoptRef(new DedicatedWorkerContext(url, userAgent, settings, thread, topOrigin));
@@ -54,12 +53,15 @@ DedicatedWorkerContext::DedicatedWorkerContext(const KURL& url, const String& us
 {
 }
 
+DedicatedWorkerContext::~DedicatedWorkerContext()
+{
+}
+
 const AtomicString& DedicatedWorkerContext::interfaceName() const
 {
     return eventNames().interfaceForDedicatedWorkerContext;
 }
 
-// FIXME: remove this when we update the ObjC bindings (bug #28774).
 void DedicatedWorkerContext::postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort* port, ExceptionCode& ec)
 {
     MessagePortArray ports;
@@ -68,11 +70,6 @@ void DedicatedWorkerContext::postMessage(PassRefPtr<SerializedScriptValue> messa
     postMessage(message, &ports, ec);
 }
 
-void DedicatedWorkerContext::postMessage(PassRefPtr<SerializedScriptValue> message, ExceptionCode& ec)
-{
-    postMessage(message, static_cast<MessagePortArray*>(0), ec);
-}
-
 void DedicatedWorkerContext::postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray* ports, ExceptionCode& ec)
 {
     // Disentangle the port in preparation for sending it to the remote context.
index 12f7ca2..18642c5 100644 (file)
@@ -45,23 +45,24 @@ namespace WebCore {
     public:
         typedef WorkerContext Base;
         static PassRefPtr<DedicatedWorkerContext> create(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, DedicatedWorkerThread*, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType contentSecurityPolicyType, PassRefPtr<SecurityOrigin> topOrigin);
+        virtual ~DedicatedWorkerContext();
 
-        virtual bool isDedicatedWorkerContext() const { return true; }
+        virtual bool isDedicatedWorkerContext() const OVERRIDE { return true; }
 
         // Overridden to allow us to check our pending activity after executing imported script.
-        virtual void importScripts(const Vector<String>& urls, ExceptionCode&);
+        virtual void importScripts(const Vector<String>& urls, ExceptionCode&) OVERRIDE;
 
         // EventTarget
-        virtual const AtomicString& interfaceName() const;
+        virtual const AtomicString& interfaceName() const OVERRIDE;
 
-        void postMessage(PassRefPtr<SerializedScriptValue>, ExceptionCode&);
         void postMessage(PassRefPtr<SerializedScriptValue>, const MessagePortArray*, ExceptionCode&);
-        // FIXME: remove this when we update the ObjC bindings (bug #28774).
+        // Needed for Objective-C bindings (see bug 28774).
         void postMessage(PassRefPtr<SerializedScriptValue>, MessagePort*, ExceptionCode&);
 
         DEFINE_ATTRIBUTE_EVENT_LISTENER(message);
 
         DedicatedWorkerThread* thread();
+
     private:
         DedicatedWorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, DedicatedWorkerThread*, PassRefPtr<SecurityOrigin> topOrigin);
     };
index 5d94072..2bbb31b 100644 (file)
@@ -43,11 +43,11 @@ namespace WebCore {
     public:
         static PassRefPtr<DedicatedWorkerThread> create(const KURL& scriptURL, const String& userAgent, const GroupSettings*, const String& sourceCode, WorkerLoaderProxy&, WorkerObjectProxy&, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, const SecurityOrigin* topOrigin);
         WorkerObjectProxy& workerObjectProxy() const { return m_workerObjectProxy; }
-        ~DedicatedWorkerThread();
+        virtual ~DedicatedWorkerThread();
 
     protected:
-        virtual PassRefPtr<WorkerContext> createWorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, PassRefPtr<SecurityOrigin> topOrigin);
-        virtual void runEventLoop();
+        virtual PassRefPtr<WorkerContext> createWorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, PassRefPtr<SecurityOrigin> topOrigin) OVERRIDE;
+        virtual void runEventLoop() OVERRIDE;
 
     private:
         DedicatedWorkerThread(const KURL&, const String& userAgent, const GroupSettings*, const String& sourceCode, WorkerLoaderProxy&, WorkerObjectProxy&, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, const SecurityOrigin* topOrigin);
index 845aed9..5202cc7 100644 (file)
@@ -45,13 +45,11 @@ namespace WebCore {
 
         MessagePort* port() const { return m_port.get(); }
 
-        virtual const AtomicString& interfaceName() const;
+        virtual const AtomicString& interfaceName() const OVERRIDE;
 
     private:
         explicit SharedWorker(ScriptExecutionContext*);
 
-        virtual SharedWorker* toSharedWorker() { return this; }
-
         RefPtr<MessagePort> m_port;
     };
 
index 87ebecb..c7442d2 100644 (file)
@@ -47,19 +47,20 @@ namespace WebCore {
         static PassRefPtr<SharedWorkerContext> create(const String& name, const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, SharedWorkerThread*, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType contentSecurityPolicyType);
         virtual ~SharedWorkerContext();
 
-        virtual bool isSharedWorkerContext() const { return true; }
+        virtual bool isSharedWorkerContext() const OVERRIDE { return true; }
 
         // EventTarget
-        virtual const AtomicString& interfaceName() const;
+        virtual const AtomicString& interfaceName() const OVERRIDE;
 
         // Setters/Getters for attributes in SharedWorkerContext.idl
         DEFINE_ATTRIBUTE_EVENT_LISTENER(connect);
         String name() const { return m_name; }
 
         SharedWorkerThread* thread();
+
     private:
         SharedWorkerContext(const String& name, const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, SharedWorkerThread*);
-        virtual void logExceptionToConsole(const String& errorMessage, const String& sourceURL, int lineNumber, PassRefPtr<ScriptCallStack>);
+        virtual void logExceptionToConsole(const String& errorMessage, const String& sourceURL, int lineNumber, PassRefPtr<ScriptCallStack>) OVERRIDE;
 
         String m_name;
     };
index c383313..c8dcc18 100644 (file)
@@ -40,10 +40,10 @@ namespace WebCore {
     class SharedWorkerThread : public WorkerThread {
     public:
         static PassRefPtr<SharedWorkerThread> create(const String& name, const KURL&, const String& userAgent, const GroupSettings*, const String& sourceCode, WorkerLoaderProxy&, WorkerReportingProxy&, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType);
-        ~SharedWorkerThread();
+        virtual ~SharedWorkerThread();
 
     protected:
-        virtual PassRefPtr<WorkerContext> createWorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, PassRefPtr<SecurityOrigin> topOrigin);
+        virtual PassRefPtr<WorkerContext> createWorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType, PassRefPtr<SecurityOrigin> topOrigin) OVERRIDE;
 
     private:
         SharedWorkerThread(const String& name, const KURL&, const String& userAgent, const GroupSettings*, const String& sourceCode, WorkerLoaderProxy&, WorkerReportingProxy&, WorkerThreadStartMode, const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType);
index f79a8b0..e3fc7fb 100644 (file)
@@ -94,7 +94,6 @@ const AtomicString& Worker::interfaceName() const
     return eventNames().interfaceForWorker;
 }
 
-// FIXME: remove this when we update the ObjC bindings (bug #28774).
 void Worker::postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort* port, ExceptionCode& ec)
 {
     MessagePortArray ports;
@@ -103,11 +102,6 @@ void Worker::postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort*
     postMessage(message, &ports, ec);
 }
 
-void Worker::postMessage(PassRefPtr<SerializedScriptValue> message, ExceptionCode& ec)
-{
-    postMessage(message, static_cast<MessagePortArray*>(0), ec);
-}
-
 void Worker::postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray* ports, ExceptionCode& ec)
 {
     // Disentangle the port in preparation for sending it to the remote context.
index c7a1f8d..e329604 100644 (file)
@@ -54,18 +54,17 @@ namespace WebCore {
         static PassRefPtr<Worker> create(ScriptExecutionContext*, const String& url, ExceptionCode&);
         virtual ~Worker();
 
-        virtual const AtomicString& interfaceName() const;
+        virtual const AtomicString& interfaceName() const OVERRIDE;
 
-        void postMessage(PassRefPtr<SerializedScriptValue> message, ExceptionCode&);
         void postMessage(PassRefPtr<SerializedScriptValue> message, const MessagePortArray*, ExceptionCode&);
-        // FIXME: remove this when we update the ObjC bindings (bug #28774).
+        // Needed for Objective-C bindings (see bug 28774).
         void postMessage(PassRefPtr<SerializedScriptValue> message, MessagePort*, ExceptionCode&);
 
         void terminate();
 
-        virtual bool canSuspend() const;
-        virtual void stop();
-        virtual bool hasPendingActivity() const;
+        virtual bool canSuspend() const OVERRIDE;
+        virtual void stop() OVERRIDE;
+        virtual bool hasPendingActivity() const OVERRIDE;
     
         DEFINE_ATTRIBUTE_EVENT_LISTENER(message);
 
@@ -73,11 +72,11 @@ namespace WebCore {
         explicit Worker(ScriptExecutionContext*);
 
         // WorkerScriptLoaderClient callbacks
-        virtual void didReceiveResponse(unsigned long identifier, const ResourceResponse&);
-        virtual void notifyFinished();
+        virtual void didReceiveResponse(unsigned long identifier, const ResourceResponse&) OVERRIDE;
+        virtual void notifyFinished() OVERRIDE;
 
-        virtual void refEventTarget() { ref(); }
-        virtual void derefEventTarget() { deref(); }
+        virtual void refEventTarget() OVERRIDE { ref(); }
+        virtual void derefEventTarget() OVERRIDE { deref(); }
 
         RefPtr<WorkerScriptLoader> m_scriptLoader;
         WorkerContextProxy* m_contextProxy; // The proxy outlives the worker to perform thread shutdown.
index 7b0ee1e..5e175b8 100644 (file)
@@ -59,9 +59,9 @@ namespace WebCore {
     public:
         virtual ~WorkerContext();
 
-        virtual bool isWorkerContext() const { return true; }
+        virtual bool isWorkerContext() const OVERRIDE { return true; }
 
-        virtual ScriptExecutionContext* scriptExecutionContext() const;
+        virtual ScriptExecutionContext* scriptExecutionContext() const OVERRIDE;
 
         virtual bool isSharedWorkerContext() const { return false; }
         virtual bool isDedicatedWorkerContext() const { return false; }
@@ -72,7 +72,7 @@ namespace WebCore {
         const GroupSettings* groupSettings() { return m_groupSettings.get(); }
         virtual String userAgent(const KURL&) const;
 
-        virtual void disableEval(const String& errorMessage);
+        virtual void disableEval(const String& errorMessage) OVERRIDE;
 
         WorkerScriptController* script() { return m_script.get(); }
         void clearScript() { m_script.clear(); }
@@ -84,7 +84,7 @@ namespace WebCore {
 
         bool hasPendingActivity() const;
 
-        virtual void postTask(PassOwnPtr<Task>); // Executes the task on context's thread asynchronously.
+        virtual void postTask(PassOwnPtr<Task>) OVERRIDE; // Executes the task on context's thread asynchronously.
 
         // WorkerGlobalScope
         WorkerContext* self() { return this; }
@@ -104,10 +104,10 @@ namespace WebCore {
         void clearInterval(int timeoutId);
 
         // ScriptExecutionContext
-        virtual WorkerEventQueue* eventQueue() const;
+        virtual WorkerEventQueue* eventQueue() const OVERRIDE;
 
-        virtual bool isContextThread() const;
-        virtual bool isJSExecutionForbidden() const;
+        virtual bool isContextThread() const OVERRIDE;
+        virtual bool isJSExecutionForbidden() const OVERRIDE;
 
 #if ENABLE(INSPECTOR)
         WorkerInspectorController* workerInspectorController() { return m_workerInspectorController.get(); }
@@ -144,25 +144,25 @@ namespace WebCore {
         WorkerContext(const KURL&, const String& userAgent, PassOwnPtr<GroupSettings>, WorkerThread*, PassRefPtr<SecurityOrigin> topOrigin);
         void applyContentSecurityPolicyFromString(const String& contentSecurityPolicy, ContentSecurityPolicy::HeaderType);
 
-        virtual void logExceptionToConsole(const String& errorMessage, const String& sourceURL, int lineNumber, PassRefPtr<ScriptCallStack>);
+        virtual void logExceptionToConsole(const String& errorMessage, const String& sourceURL, int lineNumber, PassRefPtr<ScriptCallStack>) OVERRIDE;
         void addMessageToWorkerConsole(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, PassRefPtr<ScriptCallStack>, ScriptState* = 0, unsigned long requestIdentifier = 0);
 
     private:
-        virtual void refScriptExecutionContext() { ref(); }
-        virtual void derefScriptExecutionContext() { deref(); }
+        virtual void refScriptExecutionContext() OVERRIDE { ref(); }
+        virtual void derefScriptExecutionContext() OVERRIDE { deref(); }
 
-        virtual void refEventTarget() { ref(); }
-        virtual void derefEventTarget() { deref(); }
-        virtual EventTargetData* eventTargetData();
-        virtual EventTargetData* ensureEventTargetData();
+        virtual void refEventTarget() OVERRIDE { ref(); }
+        virtual void derefEventTarget() OVERRIDE { deref(); }
+        virtual EventTargetData* eventTargetData() OVERRIDE;
+        virtual EventTargetData* ensureEventTargetData() OVERRIDE;
 
-        virtual const KURL& virtualURL() const;
+        virtual const KURL& virtualURL() const OVERRIDE;
         virtual KURL virtualCompleteURL(const String&) const;
 
-        virtual void addMessage(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, PassRefPtr<ScriptCallStack>, ScriptState* = 0, unsigned long requestIdentifier = 0);
-        virtual void addConsoleMessage(MessageSource, MessageLevel, const String& message, unsigned long requestIdentifier = 0);
+        virtual void addMessage(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, PassRefPtr<ScriptCallStack>, ScriptState* = 0, unsigned long requestIdentifier = 0) OVERRIDE;
+        virtual void addConsoleMessage(MessageSource, MessageLevel, const String& message, unsigned long requestIdentifier = 0) OVERRIDE;
 
-        virtual EventTarget* errorEventTarget();
+        virtual EventTarget* errorEventTarget() OVERRIDE;
 
         KURL m_url;
         String m_userAgent;
index d8fb699..0461b69 100644 (file)
@@ -53,36 +53,36 @@ namespace WebCore {
 
         // Implementations of WorkerContextProxy.
         // (Only use these methods in the worker object thread.)
-        virtual void startWorkerContext(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode);
-        virtual void terminateWorkerContext();
-        virtual void postMessageToWorkerContext(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>);
-        virtual bool hasPendingActivity() const;
-        virtual void workerObjectDestroyed();
+        virtual void startWorkerContext(const KURL& scriptURL, const String& userAgent, const String& sourceCode, WorkerThreadStartMode) OVERRIDE;
+        virtual void terminateWorkerContext() OVERRIDE;
+        virtual void postMessageToWorkerContext(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>) OVERRIDE;
+        virtual bool hasPendingActivity() const OVERRIDE;
+        virtual void workerObjectDestroyed() OVERRIDE;
 #if ENABLE(INSPECTOR)
-        virtual void connectToInspector(WorkerContextProxy::PageInspector*);
-        virtual void disconnectFromInspector();
-        virtual void sendMessageToInspector(const String&);
+        virtual void connectToInspector(WorkerContextProxy::PageInspector*) OVERRIDE;
+        virtual void disconnectFromInspector() OVERRIDE;
+        virtual void sendMessageToInspector(const String&) OVERRIDE;
 #endif
 
         // Implementations of WorkerObjectProxy.
         // (Only use these methods in the worker context thread.)
-        virtual void postMessageToWorkerObject(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>);
-        virtual void postExceptionToWorkerObject(const String& errorMessage, int lineNumber, const String& sourceURL);
-        virtual void postConsoleMessageToWorkerObject(MessageSource, MessageLevel, const String& message, int lineNumber, const String& sourceURL);
+        virtual void postMessageToWorkerObject(PassRefPtr<SerializedScriptValue>, PassOwnPtr<MessagePortChannelArray>) OVERRIDE;
+        virtual void postExceptionToWorkerObject(const String& errorMessage, int lineNumber, const String& sourceURL) OVERRIDE;
+        virtual void postConsoleMessageToWorkerObject(MessageSource, MessageLevel, const String& message, int lineNumber, const String& sourceURL) OVERRIDE;
 #if ENABLE(INSPECTOR)
-        virtual void postMessageToPageInspector(const String&);
-        virtual void updateInspectorStateCookie(const String&);
+        virtual void postMessageToPageInspector(const String&) OVERRIDE;
+        virtual void updateInspectorStateCookie(const String&) OVERRIDE;
 #endif
-        virtual void confirmMessageFromWorkerObject(bool hasPendingActivity);
-        virtual void reportPendingActivity(bool hasPendingActivity);
-        virtual void workerContextClosed();
-        virtual void workerContextDestroyed();
+        virtual void confirmMessageFromWorkerObject(bool hasPendingActivity) OVERRIDE;
+        virtual void reportPendingActivity(bool hasPendingActivity) OVERRIDE;
+        virtual void workerContextClosed() OVERRIDE;
+        virtual void workerContextDestroyed() OVERRIDE;
 
         // Implementation of WorkerLoaderProxy.
         // These methods are called on different threads to schedule loading
         // requests and to send callbacks back to WorkerContext.
-        virtual void postTaskToLoader(PassOwnPtr<ScriptExecutionContext::Task>);
-        virtual bool postTaskForModeToWorkerContext(PassOwnPtr<ScriptExecutionContext::Task>, const String& mode);
+        virtual void postTaskToLoader(PassOwnPtr<ScriptExecutionContext::Task>) OVERRIDE;
+        virtual bool postTaskForModeToWorkerContext(PassOwnPtr<ScriptExecutionContext::Task>, const String& mode) OVERRIDE;
 
         void workerThreadCreated(PassRefPtr<DedicatedWorkerThread>);
 
index 33e0bc3..c7a16e5 100644 (file)
@@ -50,7 +50,7 @@ namespace WebCore {
         virtual void reportPendingActivity(bool hasPendingActivity) = 0;
 
         // No need to notify the parent page context when dedicated workers are closing.
-        virtual void workerContextClosed() { }
+        virtual void workerContextClosed() OVERRIDE { }
     };
 
 } // namespace WebCore
index 1fe5689..9a5d593 100644 (file)
@@ -67,11 +67,11 @@ namespace WebCore {
         bool failed() const { return m_failed; }
         unsigned long identifier() const { return m_identifier; }
 
-        virtual void didReceiveResponse(unsigned long /*identifier*/, const ResourceResponse&);
-        virtual void didReceiveData(const char* data, int dataLength);
-        virtual void didFinishLoading(unsigned long identifier, double);
-        virtual void didFail(const ResourceError&);
-        virtual void didFailRedirectCheck();
+        virtual void didReceiveResponse(unsigned long /*identifier*/, const ResourceResponse&) OVERRIDE;
+        virtual void didReceiveData(const char* data, int dataLength) OVERRIDE;
+        virtual void didFinishLoading(unsigned long identifier, double) OVERRIDE;
+        virtual void didFail(const ResourceError&) OVERRIDE;
+        virtual void didFailRedirectCheck() OVERRIDE;
 
 #if PLATFORM(CHROMIUM) || PLATFORM(BLACKBERRY)
         void setTargetType(ResourceRequest::TargetType targetType) { m_targetType = targetType; }