Diff
Modified: tags/Safari-538.26/LayoutTests/ChangeLog (166538 => 166539)
--- tags/Safari-538.26/LayoutTests/ChangeLog 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/LayoutTests/ChangeLog 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,3 +1,18 @@
+2014-03-31 Matthew Hanson <[email protected]>
+
+ Rollout r166415.
+
+ 2014-03-28 Michael Saboff <[email protected]>
+
+ Unreviewed, rolling r166248 back in.
+
+ Turns out r166070 didn't cause a 2% performance loss in page load times
+
+ Reverted changeset:
+
+ Unreviewed, rolling out r166126.
+ Rollout r166126 in prepartion to roll out prerequisite r166070
+
2014-03-30 Alexey Proskuryakov <[email protected]>
REGRESSION: compositing/columns/composited-rl-paginated-repaint.html is flaky
Deleted: tags/Safari-538.26/LayoutTests/fast/workers/resources/worker-strict.js (166538 => 166539)
--- tags/Safari-538.26/LayoutTests/fast/workers/resources/worker-strict.js 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/LayoutTests/fast/workers/resources/worker-strict.js 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,11 +0,0 @@
-"use strict";
-
-postMessage("SUCCESS: postMessage() called directly");
-
-(function () {
- this.done = function() { return "SUCCESS: called function via attribute on WorkerGlobalScope"; };
-
- this.postMessage(this.done());
-
- this.postMessage("DONE");
-}).call(this);
Deleted: tags/Safari-538.26/LayoutTests/fast/workers/worker-strict-expected.txt (166538 => 166539)
--- tags/Safari-538.26/LayoutTests/fast/workers/worker-strict-expected.txt 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/LayoutTests/fast/workers/worker-strict-expected.txt 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,6 +0,0 @@
-Test calling functions in strict mode script from WorkerContext. Should print SUCCESS lines, followed by DONE.
-
-SUCCESS: postMessage() called directly
-SUCCESS: called function via attribute on WorkerGlobalScope
-DONE
-
Deleted: tags/Safari-538.26/LayoutTests/fast/workers/worker-strict.html (166538 => 166539)
--- tags/Safari-538.26/LayoutTests/fast/workers/worker-strict.html 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/LayoutTests/fast/workers/worker-strict.html 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,25 +0,0 @@
-<body>
-<p>Test calling functions in strict mode script from WorkerContext. Should print SUCCESS lines, followed by DONE.</p>
-<div id=result></div>
-<script>
-
-if (window.testRunner) {
- testRunner.dumpAsText();
- testRunner.waitUntilDone();
-}
-
-function log(message)
-{
- document.getElementById("result").innerHTML += message + "<br>";
-}
-
-var worker = new Worker("resources/worker-strict.js");
-worker._onmessage_ = function(event) {
- log(event.data);
- if (event.data == "DONE") {
- if (window.testRunner)
- testRunner.notifyDone();
- }
-};
-</script>
-</body>
Modified: tags/Safari-538.26/Source/_javascript_Core/API/JSContextRef.cpp (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/API/JSContextRef.cpp 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/API/JSContextRef.cpp 2014-03-31 22:29:36 UTC (rev 166539)
@@ -129,6 +129,7 @@
if (!globalObjectClass) {
JSGlobalObject* globalObject = JSGlobalObject::create(*vm, JSGlobalObject::createStructure(*vm, jsNull()));
+ globalObject->setGlobalThis(*vm, JSProxy::create(*vm, JSProxy::createStructure(*vm, globalObject, globalObject->prototype()), globalObject));
return JSGlobalContextRetain(toGlobalRef(globalObject->globalExec()));
}
Modified: tags/Safari-538.26/Source/_javascript_Core/API/tests/testapi.c (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/API/tests/testapi.c 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/API/tests/testapi.c 2014-03-31 22:29:36 UTC (rev 166539)
@@ -891,8 +891,10 @@
// Ensure that an execution context is passed in
ASSERT(context);
+ // Ensure that the global object is set to the object that we were passed
JSObjectRef globalObject = JSContextGetGlobalObject(context);
ASSERT(globalObject);
+ ASSERT(object == globalObject);
// Ensure that the standard global properties have been set on the global object
JSStringRef array = JSStringCreateWithUTF8CString("Array");
Modified: tags/Safari-538.26/Source/_javascript_Core/ChangeLog (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/ChangeLog 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/ChangeLog 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,3 +1,18 @@
+2014-03-31 Matthew Hanson <[email protected]>
+
+ Rollout r166415.
+
+ 2014-03-28 Michael Saboff <[email protected]>
+
+ Unreviewed, rolling r166248 back in.
+
+ Turns out r166070 didn't cause a 2% performance loss in page load times
+
+ Reverted changeset:
+
+ Unreviewed, rolling out r166126.
+ Rollout r166126 in prepartion to roll out prerequisite r166070
+
2014-03-28 Filip Pizlo <[email protected]>
Land the stackmap register liveness glue with the uses of the liveness disabled
Modified: tags/Safari-538.26/Source/_javascript_Core/jsc.cpp (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/jsc.cpp 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/jsc.cpp 2014-03-31 22:29:36 UTC (rev 166539)
@@ -351,6 +351,7 @@
GlobalObject* object = new (NotNull, allocateCell<GlobalObject>(vm.heap)) GlobalObject(vm, structure);
object->finishCreation(vm, arguments);
vm.heap.addFinalizer(object, destroy);
+ object->setGlobalThis(vm, JSProxy::create(vm, JSProxy::createStructure(vm, object, object->prototype()), object));
return object;
}
Modified: tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.cpp (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.cpp 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.cpp 2014-03-31 22:29:36 UTC (rev 166539)
@@ -197,10 +197,11 @@
m_globalThis.set(vm, this, globalThis);
}
-void JSGlobalObject::init()
+void JSGlobalObject::init(JSObject* thisValue)
{
ASSERT(vm().currentThreadIsHoldingAPILock());
+ setGlobalThis(vm(), thisValue);
JSGlobalObject::globalExec()->init(0, 0, this, CallFrame::noCaller(), 0, 0);
m_debugger = 0;
Modified: tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.h (166538 => 166539)
--- tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.h 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/_javascript_Core/runtime/JSGlobalObject.h 2014-03-31 22:29:36 UTC (rev 166539)
@@ -27,7 +27,6 @@
#include "JSArray.h"
#include "JSArrayBufferPrototype.h"
#include "JSClassRef.h"
-#include "JSProxy.h"
#include "JSSegmentedVariableObject.h"
#include "JSWeakObjectMapRefInternal.h"
#include "NumberPrototype.h"
@@ -297,8 +296,7 @@
Base::finishCreation(vm);
structure()->setGlobalObject(vm, this);
m_experimentsEnabled = m_globalObjectMethodTable->_javascript_ExperimentsEnabled(this);
- init();
- setGlobalThis(vm, JSProxy::create(vm, JSProxy::createStructure(vm, this, prototype()), this));
+ init(this);
}
void finishCreation(VM& vm, JSObject* thisValue)
@@ -306,8 +304,7 @@
Base::finishCreation(vm);
structure()->setGlobalObject(vm, this);
m_experimentsEnabled = m_globalObjectMethodTable->_javascript_ExperimentsEnabled(this);
- init();
- setGlobalThis(vm, thisValue);
+ init(thisValue);
}
struct NewGlobalVar {
@@ -540,6 +537,7 @@
VM& vm() const { return m_vm; }
JSObject* globalThis() const;
+ JS_EXPORT_PRIVATE void setGlobalThis(VM&, JSObject* globalThis);
static Structure* createStructure(VM& vm, JSValue prototype)
{
@@ -592,11 +590,9 @@
private:
friend class LLIntOffsetsExtractor;
-
- JS_EXPORT_PRIVATE void setGlobalThis(VM&, JSObject* globalThis);
-
+
// FIXME: Fold reset into init.
- JS_EXPORT_PRIVATE void init();
+ JS_EXPORT_PRIVATE void init(JSObject* thisValue);
void reset(JSValue prototype);
void createThrowTypeError(VM&);
Modified: tags/Safari-538.26/Source/WebCore/ChangeLog (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/ChangeLog 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/ChangeLog 2014-03-31 22:29:36 UTC (rev 166539)
@@ -1,3 +1,18 @@
+2014-03-31 Matthew Hanson <[email protected]>
+
+ Rollout r166415.
+
+ 2014-03-28 Michael Saboff <[email protected]>
+
+ Unreviewed, rolling r166248 back in.
+
+ Turns out r166070 didn't cause a 2% performance loss in page load times
+
+ Reverted changeset:
+
+ Unreviewed, rolling out r166126.
+ Rollout r166126 in prepartion to roll out prerequisite r166070
+
2014-03-30 Jinwoo Song <[email protected]>
Adopt range-based for loops to TextCheckerEnchant
Modified: tags/Safari-538.26/Source/WebCore/bindings/js/JSWorkerGlobalScopeBase.cpp (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/bindings/js/JSWorkerGlobalScopeBase.cpp 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/bindings/js/JSWorkerGlobalScopeBase.cpp 2014-03-31 22:29:36 UTC (rev 166539)
@@ -130,8 +130,6 @@
const ClassInfo* classInfo = asObject(value)->classInfo();
if (classInfo == JSDedicatedWorkerGlobalScope::info())
return jsCast<JSDedicatedWorkerGlobalScope*>(asObject(value));
- if (classInfo == JSProxy::info())
- return jsDynamicCast<JSDedicatedWorkerGlobalScope*>(jsCast<JSProxy*>(asObject(value))->target());
return 0;
}
@@ -143,8 +141,6 @@
const ClassInfo* classInfo = asObject(value)->classInfo();
if (classInfo == JSSharedWorkerGlobalScope::info())
return jsCast<JSSharedWorkerGlobalScope*>(asObject(value));
- if (classInfo == JSProxy::info())
- return jsDynamicCast<JSSharedWorkerGlobalScope*>(jsCast<JSProxy*>(asObject(value))->target());
return 0;
}
#endif
Modified: tags/Safari-538.26/Source/WebCore/bindings/js/WorkerScriptController.cpp (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/bindings/js/WorkerScriptController.cpp 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/bindings/js/WorkerScriptController.cpp 2014-03-31 22:29:36 UTC (rev 166539)
@@ -132,7 +132,7 @@
JSLockHolder lock(exec);
JSValue evaluationException;
- JSC::evaluate(exec, sourceCode.jsSourceCode(), m_workerGlobalScopeWrapper->globalThis(), &evaluationException);
+ JSC::evaluate(exec, sourceCode.jsSourceCode(), m_workerGlobalScopeWrapper.get(), &evaluationException);
if ((evaluationException && isTerminatedExecutionException(evaluationException)) || m_workerGlobalScopeWrapper->vm().watchdog.didFire()) {
forbidExecution();
Modified: tags/Safari-538.26/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl 2014-03-31 22:29:36 UTC (rev 166539)
@@ -30,7 +30,6 @@
[
GlobalContext=DedicatedWorkerGlobalScope,
- CustomProxyToJSObject,
JSGenerateToNativeObject,
JSNoStaticTables,
] interface DedicatedWorkerGlobalScope : WorkerGlobalScope {
Modified: tags/Safari-538.26/Source/WebCore/workers/SharedWorkerGlobalScope.idl (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/workers/SharedWorkerGlobalScope.idl 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/workers/SharedWorkerGlobalScope.idl 2014-03-31 22:29:36 UTC (rev 166539)
@@ -30,7 +30,6 @@
[
GlobalContext=SharedWorkerGlobalScope,
- CustomProxyToJSObject,
Conditional=SHARED_WORKERS,
JSGenerateToNativeObject,
JSNoStaticTables,
Modified: tags/Safari-538.26/Source/WebCore/workers/WorkerGlobalScope.idl (166538 => 166539)
--- tags/Safari-538.26/Source/WebCore/workers/WorkerGlobalScope.idl 2014-03-31 22:28:35 UTC (rev 166538)
+++ tags/Safari-538.26/Source/WebCore/workers/WorkerGlobalScope.idl 2014-03-31 22:29:36 UTC (rev 166539)
@@ -26,7 +26,6 @@
[
GlobalContext=WorkerGlobalScope,
- CustomProxyToJSObject,
JSCustomMarkFunction,
JSCustomGetOwnPropertySlotAndDescriptor,
EventTarget,
