Unreviewed, rolling out r244900.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 May 2019 18:42:16 +0000 (18:42 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 May 2019 18:42:16 +0000 (18:42 +0000)
Caused media/track/track-cue-missing.html to fail on debug
queues

Reverted changeset:

"Add logging for RenderLayer clip rects"
https://bugs.webkit.org/show_bug.cgi?id=197547
https://trac.webkit.org/changeset/244900

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

Source/WebCore/ChangeLog
Source/WebCore/platform/Logging.h
Source/WebCore/rendering/ClipRect.cpp
Source/WebCore/rendering/ClipRect.h
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderLayer.h

index 26177d0..a421106 100644 (file)
@@ -1,3 +1,16 @@
+2019-05-07  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r244900.
+
+        Caused media/track/track-cue-missing.html to fail on debug
+        queues
+
+        Reverted changeset:
+
+        "Add logging for RenderLayer clip rects"
+        https://bugs.webkit.org/show_bug.cgi?id=197547
+        https://trac.webkit.org/changeset/244900
+
 2019-05-07  Antti Koivisto  <antti@apple.com>
 
         <body> with overflow:hidden CSS is scrollable on iOS
index e120a29..0171229 100644 (file)
@@ -42,7 +42,6 @@ namespace WebCore {
     M(Animations) \
     M(ApplePay) \
     M(Archives) \
-    M(ClipRects) \
     M(Compositing) \
     M(ContentFiltering) \
     M(ContentObservation) \
index 3630d16..b4db563 100644 (file)
@@ -38,17 +38,4 @@ bool ClipRect::intersects(const HitTestLocation& hitTestLocation) const
     return hitTestLocation.intersects(m_rect);
 }
 
-TextStream& operator<<(TextStream& ts, const ClipRect& clipRect)
-{
-    ts << "rect ";
-    if (clipRect.isInfinite())
-        ts << "infinite";
-    else
-        ts << clipRect.rect();
-
-    if (clipRect.affectedByRadius())
-        ts << " affected by radius";
-    return ts;
-}
-
 }
index 52abb67..5ed7a6c 100644 (file)
 
 #include "LayoutRect.h"
 
-namespace WTF {
-class TextStream;
-}
-
 namespace WebCore {
 
 class HitTestLocation;
@@ -107,6 +103,4 @@ inline ClipRect intersection(const ClipRect& a, const ClipRect& b)
     return c;
 }
 
-WTF::TextStream& operator<<(WTF::TextStream&, const ClipRect&);
-
 } // namespace WebCore
index 21c28c3..a3ff520 100644 (file)
@@ -268,19 +268,6 @@ void makeMatrixRenderable(TransformationMatrix& matrix, bool has3DRendering)
 #endif
 }
 
-#if !LOG_DISABLED
-static TextStream& operator<<(TextStream& ts, const ClipRects& clipRects)
-{
-    TextStream::GroupScope scope(ts);
-    ts << indent << "ClipRects\n";
-    ts << indent << "  overflow  : " << clipRects.overflowClipRect() << "\n";
-    ts << indent << "  fixed     : " << clipRects.fixedClipRect() << "\n";
-    ts << indent << "  positioned: " << clipRects.posClipRect() << "\n";
-
-    return ts;
-}
-#endif
-
 RenderLayer::RenderLayer(RenderLayerModelObject& rendererLayerModelObject)
     : m_isRenderViewLayer(rendererLayerModelObject.isRenderView())
     , m_forcedStackingContext(rendererLayerModelObject.isMedia())
@@ -5541,8 +5528,6 @@ void RenderLayer::calculateClipRects(const ClipRectsContext& clipRectsContext, C
             clipRects.setFixedClipRect(intersection(newPosClip, clipRects.fixedClipRect()));
         }
     }
-    
-    LOG_WITH_STREAM(ClipRects, stream << "RenderLayer " << this << " calculateClipRects " << clipRects);
 }
 
 Ref<ClipRects> RenderLayer::parentClipRects(const ClipRectsContext& clipRectsContext) const
@@ -5592,8 +5577,6 @@ ClipRect RenderLayer::backgroundClipRect(const ClipRectsContext& clipRectsContex
     // Note: infinite clipRects should not be scrolled here, otherwise they will accidentally no longer be considered infinite.
     if (parentRects->fixed() && &clipRectsContext.rootLayer->renderer() == &view && !backgroundClipRect.isInfinite())
         backgroundClipRect.moveBy(view.frameView().scrollPositionForFixedPosition());
-
-    LOG_WITH_STREAM(ClipRects, stream << "RenderLayer " << this << " backgroundClipRect with context " << clipRectsContext << " returning " << backgroundClipRect);
     return backgroundClipRect;
 }
 
@@ -6756,21 +6739,6 @@ void RenderLayer::invalidateEventRegion()
 #endif
 }
 
-TextStream& operator<<(WTF::TextStream& ts, ClipRectsType clipRectsType)
-{
-    switch (clipRectsType) {
-    case PaintingClipRects: ts << "painting"; break;
-    case RootRelativeClipRects: ts << "root-relative"; break;
-    case AbsoluteClipRects: ts << "absolute"; break;
-    case TemporaryClipRects: ts << "temporary"; break;
-    case NumCachedClipRectsTypes:
-    case AllClipRectTypes:
-        ts << "?";
-        break;
-    }
-    return ts;
-}
-
 TextStream& operator<<(TextStream& ts, const RenderLayer& layer)
 {
     ts << "RenderLayer " << &layer << " " << layer.size();
@@ -6789,15 +6757,6 @@ TextStream& operator<<(TextStream& ts, const RenderLayer& layer)
     return ts;
 }
 
-TextStream& operator<<(TextStream& ts, const RenderLayer::ClipRectsContext& context)
-{
-    ts.dumpProperty("root layer:", context.rootLayer);
-    ts.dumpProperty("type:", context.clipRectsType);
-    ts.dumpProperty("overflow-clip:", context.respectOverflowClip == IgnoreOverflowClip ? "ignore" : "respect");
-    
-    return ts;
-}
-
 } // namespace WebCore
 
 #if ENABLE(TREE_DEBUGGING)
index fee5060..7aad196 100644 (file)
@@ -1369,9 +1369,7 @@ void makeMatrixRenderable(TransformationMatrix&, bool has3DRendering);
 
 bool compositedWithOwnBackingStore(const RenderLayer&);
 
-WTF::TextStream& operator<<(WTF::TextStream&, ClipRectsType);
 WTF::TextStream& operator<<(WTF::TextStream&, const RenderLayer&);
-WTF::TextStream& operator<<(WTF::TextStream&, const RenderLayer::ClipRectsContext&);
 
 } // namespace WebCore