Attachment 369682 Details for Bug 174272 – [Patch] Extra Logging

[patch] [Patch] Extra Logging

174272.diff (text/plain), 6.49 KB, created by Devin Rousso on 2019-05-12 13:10:04 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Devin Rousso

Created: 2019-05-12 13:10:04 PDT

Size: 6.49 KB

patch

obsolete

>diff --git a/LayoutTests/inspector/canvas/css-canvas-clients.html b/LayoutTests/inspector/canvas/css-canvas-clients.html
>index 7fbaf6a2c20..099167d3c3a 100644
>--- a/LayoutTests/inspector/canvas/css-canvas-clients.html
>+++ b/LayoutTests/inspector/canvas/css-canvas-clients.html
>@@ -25,6 +25,8 @@ function destroyCSSCanvasClients() {
> }
> 
> function test() {
>+    InspectorTest.debug();
>+
>     let suite = InspectorTest.createAsyncSuite("Canvas.CSSCanvasClients");
> 
>     function logClientNodes(clientNodes) {
>diff --git a/LayoutTests/platform/mac/TestExpectations b/LayoutTests/platform/mac/TestExpectations
>index f273f6d3d8e..8c35987da17 100644
>--- a/LayoutTests/platform/mac/TestExpectations
>+++ b/LayoutTests/platform/mac/TestExpectations
>@@ -1031,7 +1031,6 @@ webkit.org/b/150978 fast/canvas/webgl/oes-texture-half-float-linear.html [ Pass
> webkit.org/b/173931 inspector/canvas/context-attributes.html [ Pass Timeout ]
> webkit.org/b/178028 inspector/canvas/create-context-2d.html [ Pass Timeout ]
> webkit.org/b/174066 inspector/canvas/create-context-webgl2.html [ Pass Timeout ]
>-webkit.org/b/174272 inspector/canvas/css-canvas-clients.html [ Pass Failure ]
> webkit.org/b/174066 inspector/canvas/recording-webgl2.html [ Pass Failure Timeout ]
> webkit.org/b/174066 inspector/canvas/recording-webgl2-snapshots.html [ Pass Failure Timeout ]
> webkit.org/b/174066 inspector/canvas/shaderProgram-add-remove-webgl2.html [ Pass Failure Timeout ]
>diff --git a/Source/WebCore/css/CSSImageGeneratorValue.cpp b/Source/WebCore/css/CSSImageGeneratorValue.cpp
>index ffdd3d0aa65..d0a95348b08 100644
>--- a/Source/WebCore/css/CSSImageGeneratorValue.cpp
>+++ b/Source/WebCore/css/CSSImageGeneratorValue.cpp
>@@ -68,12 +68,16 @@ CSSImageGeneratorValue::~CSSImageGeneratorValue() = default;
> 
> void CSSImageGeneratorValue::addClient(RenderElement& renderer)
> {
>+    WTFLogAlways(">>> IN -> CSSImageGeneratorValue::addClient");
>+
>     if (m_clients.isEmpty())
>         ref();
> 
>     m_clients.add(&renderer);
> 
>+    WTFLogAlways(">>> CHECK -> is<CSSCanvasValue>(this)");
>     if (is<CSSCanvasValue>(this)) {
>+        WTFLogAlways(">>> CHECK -> HTMLCanvasElement* canvasElement = downcast<CSSCanvasValue>(this)->element()");
>         if (HTMLCanvasElement* canvasElement = downcast<CSSCanvasValue>(this)->element())
>             InspectorInstrumentation::didChangeCSSCanvasClientNodes(*canvasElement);
>     }
>@@ -81,11 +85,15 @@ void CSSImageGeneratorValue::addClient(RenderElement& renderer)
> 
> void CSSImageGeneratorValue::removeClient(RenderElement& renderer)
> {
>+    WTFLogAlways(">>> IN -> CSSImageGeneratorValue::removeClient");
>+
>     ASSERT(m_clients.contains(&renderer));
>     if (!m_clients.remove(&renderer))
>         return;
> 
>+    WTFLogAlways(">>> CHECK -> is<CSSCanvasValue>(this)");
>     if (is<CSSCanvasValue>(this)) {
>+        WTFLogAlways(">>> CHECK -> HTMLCanvasElement* canvasElement = downcast<CSSCanvasValue>(this)->element()");
>         if (HTMLCanvasElement* canvasElement = downcast<CSSCanvasValue>(this)->element())
>             InspectorInstrumentation::didChangeCSSCanvasClientNodes(*canvasElement);
>     }
>diff --git a/Source/WebCore/html/CanvasBase.cpp b/Source/WebCore/html/CanvasBase.cpp
>index 9f166efdf09..6488f7c06c1 100644
>--- a/Source/WebCore/html/CanvasBase.cpp
>+++ b/Source/WebCore/html/CanvasBase.cpp
>@@ -53,16 +53,22 @@ CanvasRenderingContext* CanvasBase::renderingContext() const
> 
> void CanvasBase::addObserver(CanvasObserver& observer)
> {
>+    WTFLogAlways(">>> IN -> CanvasBase::addObserver");
>+
>     m_observers.add(&observer);
> 
>+    WTFLogAlways(">>> CHECK -> is<CSSCanvasValue::CanvasObserverProxy>(observer)");
>     if (is<CSSCanvasValue::CanvasObserverProxy>(observer))
>         InspectorInstrumentation::didChangeCSSCanvasClientNodes(*this);
> }
> 
> void CanvasBase::removeObserver(CanvasObserver& observer)
> {
>+    WTFLogAlways(">>> IN -> CanvasBase::removeObserver");
>+
>     m_observers.remove(&observer);
> 
>+    WTFLogAlways(">>> CHECK -> is<CSSCanvasValue::CanvasObserverProxy>(observer)");
>     if (is<CSSCanvasValue::CanvasObserverProxy>(observer))
>         InspectorInstrumentation::didChangeCSSCanvasClientNodes(*this);
> }
>diff --git a/Source/WebCore/inspector/InspectorInstrumentation.cpp b/Source/WebCore/inspector/InspectorInstrumentation.cpp
>index a250948c98e..3b266497ef2 100644
>--- a/Source/WebCore/inspector/InspectorInstrumentation.cpp
>+++ b/Source/WebCore/inspector/InspectorInstrumentation.cpp
>@@ -1007,6 +1007,9 @@ void InspectorInstrumentation::didSendWebSocketFrameImpl(InstrumentingAgents& in
> 
> void InspectorInstrumentation::didChangeCSSCanvasClientNodesImpl(InstrumentingAgents& instrumentingAgents, CanvasBase& canvasBase)
> {
>+    WTFLogAlways(">>> IN -> InspectorInstrumentation::didChangeCSSCanvasClientNodesImpl");
>+
>+    WTFLogAlways(">>> CHECK -> InspectorCanvasAgent* canvasAgent = instrumentingAgents.inspectorCanvasAgent()");
>     if (InspectorCanvasAgent* canvasAgent = instrumentingAgents.inspectorCanvasAgent())
>         canvasAgent->didChangeCSSCanvasClientNodes(canvasBase);
> }
>diff --git a/Source/WebCore/inspector/InspectorInstrumentation.h b/Source/WebCore/inspector/InspectorInstrumentation.h
>index c86453f1425..eb0f6b0e483 100644
>--- a/Source/WebCore/inspector/InspectorInstrumentation.h
>+++ b/Source/WebCore/inspector/InspectorInstrumentation.h
>@@ -1285,7 +1285,11 @@ inline void InspectorInstrumentation::didHandleMemoryPressure(Page& page, Critic
> 
> inline void InspectorInstrumentation::didChangeCSSCanvasClientNodes(CanvasBase& canvasBase)
> {
>+    WTFLogAlways(">>> IN -> InspectorInstrumentation::didChangeCSSCanvasClientNodes");
>+
>     FAST_RETURN_IF_NO_FRONTENDS(void());
>+
>+    WTFLogAlways(">>> CHECK -> InstrumentingAgents* instrumentingAgents = instrumentingAgentsForContext(canvasBase.scriptExecutionContext())");
>     if (InstrumentingAgents* instrumentingAgents = instrumentingAgentsForContext(canvasBase.scriptExecutionContext()))
>         didChangeCSSCanvasClientNodesImpl(*instrumentingAgents, canvasBase);
> }
>diff --git a/Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp b/Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp
>index a33b6032b7f..d08615c74ae 100644
>--- a/Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp
>+++ b/Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp
>@@ -377,6 +377,8 @@ void InspectorCanvasAgent::frameNavigated(Frame& frame)
> 
> void InspectorCanvasAgent::didChangeCSSCanvasClientNodes(CanvasBase& canvasBase)
> {
>+    WTFLogAlways(">>> IN - InspectorCanvasAgent::didChangeCSSCanvasClientNodes");
>+
>     auto* context = canvasBase.renderingContext();
>     if (!context) {
>         ASSERT_NOT_REACHED();

Flags:

hi: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 174272: 369660 | 369661 | 369665 | 369666 | 369682 | 369689 | 369691 | 369693 | 369696 | 369697 | 369701 | 369703 | 369704 | 369705 | 369706 | 369713 | 369716 | 369720 | 369758 | 369775 | 369859 | 433685