Changeset 87424 in webkit

Timestamp:

May 26, 2011 1:09:55 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-05-26 Leandro Gracia Gil <​leandrogracia@chromium.org>

Reviewed by Dmitry Titov.

[V8] Add missing compile guards for WebWorkers in WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=61535

No new tests - refactoring only.

• bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::invokeEventHandler):
• bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::instantiateV8Object):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/V8AbstractEventListener.cpp (modified) (1 diff)
• bindings/v8/V8DOMWrapper.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-05-26  Leandro Gracia Gil  <leandrogracia@chromium.org>
+
+        Reviewed by Dmitry Titov.
+
+        [V8] Add missing compile guards for WebWorkers in WebCore.
+        https://bugs.webkit.org/show_bug.cgi?id=61535
+
+        No new tests - refactoring only.
+
+        * bindings/v8/V8AbstractEventListener.cpp:
+        (WebCore::V8AbstractEventListener::invokeEventHandler):
+        * bindings/v8/V8DOMWrapper.cpp:
+        (WebCore::V8DOMWrapper::instantiateV8Object):
+
 2011-05-26  Mihai Parparita  <mihaip@chromium.org>
 

trunk/Source/WebCore/bindings/v8/V8AbstractEventListener.cpp

@@ -157 +157 @@
 
         if (!tryCatch.CanContinue()) { // Result of TerminateExecution().
+#if ENABLE(WORKERS)
             if (context->isWorkerContext())
                 static_cast<WorkerContext*>(context)->script()->forbidExecution();
+#endif
             return;
         }

trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp

@@ -257 +257 @@
 v8::Local<v8::Object> V8DOMWrapper::instantiateV8Object(V8Proxy* proxy, WrapperTypeInfo* type, void* impl)
 {
+#if ENABLE(WORKERS)
     WorkerContext* workerContext = 0;
+#endif
     if (V8IsolatedContext::getEntered()) {
         // This effectively disables the wrapper cache for isolated worlds.