Unreviewed, rolling out r250381.
authortsavell@apple.com <tsavell@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Sep 2019 17:49:47 +0000 (17:49 +0000)
committertsavell@apple.com <tsavell@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Sep 2019 17:49:47 +0000 (17:49 +0000)
broke multiple webgpu/ tests on Mojave WK2

Reverted changeset:

"Flaky Test: inspector/canvas/updateShader.html"
https://bugs.webkit.org/show_bug.cgi?id=202186
https://trac.webkit.org/changeset/250381

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

Source/WebCore/ChangeLog
Source/WebCore/Modules/webgpu/WebGPUPipeline.cpp
Source/WebCore/Modules/webgpu/WebGPUPipeline.h
Source/WebCore/html/canvas/WebGLProgram.cpp
Source/WebCore/html/canvas/WebGLProgram.h

index 3b1e781..efaaf90 100644 (file)
@@ -1,3 +1,15 @@
+2019-09-26  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, rolling out r250381.
+
+        broke multiple webgpu/ tests on Mojave WK2
+
+        Reverted changeset:
+
+        "Flaky Test: inspector/canvas/updateShader.html"
+        https://bugs.webkit.org/show_bug.cgi?id=202186
+        https://trac.webkit.org/changeset/250381
+
 2019-09-26  Alexey Shvayka  <shvaikalesh@gmail.com>
 
         Non-callable "handleEvent" property is silently ignored
index f33baf4..7a12159 100644 (file)
@@ -57,9 +57,9 @@ Lock& WebGPUPipeline::instancesMutex()
 
 WebGPUPipeline::WebGPUPipeline(WebGPUDevice& device, GPUErrorScopes& errorScopes)
     : GPUObjectBase(makeRef(errorScopes))
-    , ContextDestructionObserver(device.scriptExecutionContext())
+    , m_scriptExecutionContext(device.scriptExecutionContext())
 {
-    ASSERT(scriptExecutionContext());
+    ASSERT(m_scriptExecutionContext);
 
     {
         LockHolder lock(instancesMutex());
@@ -78,13 +78,6 @@ WebGPUPipeline::~WebGPUPipeline()
     }
 }
 
-void WebGPUPipeline::contextDestroyed()
-{
-    InspectorInstrumentation::willDestroyWebGPUPipeline(*this);
-
-    ContextDestructionObserver::contextDestroyed();
-}
-
 } // namespace WebCore
 
 #endif // ENABLE(WEBGPU)
index 729e8a2..1e323c7 100644 (file)
@@ -27,7 +27,6 @@
 
 #if ENABLE(WEBGPU)
 
-#include "ContextDestructionObserver.h"
 #include "GPUObjectBase.h"
 #include "WebGPUShaderModule.h"
 #include <wtf/Forward.h>
@@ -38,18 +37,17 @@ class ScriptExecutionContext;
 class GPUErrorScopes;
 class WebGPUDevice;
 
-class WebGPUPipeline : public GPUObjectBase, public ContextDestructionObserver {
+class WebGPUPipeline : public GPUObjectBase {
 public:
     virtual ~WebGPUPipeline();
 
     static HashMap<WebGPUPipeline*, WebGPUDevice*>& instances(const LockHolder&);
     static Lock& instancesMutex();
 
-    void contextDestroyed() final;
-
     virtual bool isRenderPipeline() const { return false; }
     virtual bool isComputePipeline() const { return false; }
 
+    ScriptExecutionContext* scriptExecutionContext() const { return m_scriptExecutionContext; }
     virtual bool isValid() const = 0;
 
     struct ShaderData {
@@ -61,6 +59,8 @@ public:
 
 protected:
     WebGPUPipeline(WebGPUDevice&, GPUErrorScopes&);
+
+    ScriptExecutionContext* m_scriptExecutionContext;
 };
 
 } // namespace WebCore
index 0850224..fe1b0c5 100644 (file)
@@ -62,9 +62,9 @@ Ref<WebGLProgram> WebGLProgram::create(WebGLRenderingContextBase& ctx)
 
 WebGLProgram::WebGLProgram(WebGLRenderingContextBase& ctx)
     : WebGLSharedObject(ctx)
-    , ContextDestructionObserver(ctx.scriptExecutionContext())
+    , m_scriptExecutionContext(ctx.scriptExecutionContext())
 {
-    ASSERT(scriptExecutionContext());
+    ASSERT(m_scriptExecutionContext);
 
     {
         LockHolder lock(instancesMutex());
@@ -87,13 +87,6 @@ WebGLProgram::~WebGLProgram()
     }
 }
 
-void WebGLProgram::contextDestroyed()
-{
-    InspectorInstrumentation::willDestroyWebGLProgram(*this);
-
-    ContextDestructionObserver::contextDestroyed();
-}
-
 void WebGLProgram::deleteObjectImpl(GraphicsContext3D* context3d, Platform3DObject obj)
 {
     context3d->deleteProgram(obj);
index 0050929..6769d1d 100644 (file)
@@ -27,7 +27,6 @@
 
 #if ENABLE(WEBGL)
 
-#include "ContextDestructionObserver.h"
 #include "WebGLSharedObject.h"
 #include <wtf/Forward.h>
 
@@ -37,7 +36,7 @@ class ScriptExecutionContext;
 class WebGLRenderingContextBase;
 class WebGLShader;
 
-class WebGLProgram final : public WebGLSharedObject, public ContextDestructionObserver {
+class WebGLProgram final : public WebGLSharedObject {
 public:
     static Ref<WebGLProgram> create(WebGLRenderingContextBase&);
     virtual ~WebGLProgram();
@@ -45,7 +44,7 @@ public:
     static HashMap<WebGLProgram*, WebGLRenderingContextBase*>& instances(const LockHolder&);
     static Lock& instancesMutex();
 
-    void contextDestroyed() final;
+    ScriptExecutionContext* scriptExecutionContext() const { return m_scriptExecutionContext; }
 
     unsigned numActiveAttribLocations();
     GC3Dint getActiveAttribLocation(GC3Duint index);
@@ -76,6 +75,8 @@ private:
     void cacheActiveAttribLocations(GraphicsContext3D*);
     void cacheInfoIfNeeded();
 
+    ScriptExecutionContext* m_scriptExecutionContext;
+
     Vector<GC3Dint> m_activeAttribLocations;
 
     GC3Dint m_linkStatus { 0 };